Skip to content

Fix getAccessToken() return type#103

Merged
miguelsanchez-upsun merged 1 commit into3.xfrom
fix-getaccesstoken-type
Mar 20, 2026
Merged

Fix getAccessToken() return type#103
miguelsanchez-upsun merged 1 commit into3.xfrom
fix-getaccesstoken-type

Conversation

@pjcdawkins
Copy link
Collaborator

Summary

  • Fix Connector::getAccessToken() return type from false|string to ?string
  • Session::get() returns null when the key is absent, but the declared type did not include null
  • The method never returned false; removing it from the union is a correction, not a behavior change

Test plan

  • Verify CLI auth with no cached session (e.g. fresh container with PLATFORMSH_CLI_TOKEN)
  • Confirm existing session-based auth still works

Generated with Claude Code

@miguelsanchez-upsun miguelsanchez-upsun merged commit 61c1a74 into 3.x Mar 20, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants