Skip to content

Improvements to registration and authorization#7

Merged
colinmollenhour merged 13 commits intomainfrom
registration-fixes
Oct 17, 2025
Merged

Improvements to registration and authorization#7
colinmollenhour merged 13 commits intomainfrom
registration-fixes

Conversation

@bob2021
Copy link
Copy Markdown
Collaborator

@bob2021 bob2021 commented Oct 2, 2025

  • Using a Number type for the invoice amount when registering via invoice throws an error for amounts like 234.00 because the trailing zeros are stripped off, so a string must be used instead.
  • Auth token is now fetched/checked just before the request is made, to help avoid using an expired token.
  • The connector's base url is now overridden for authentication requests, so a connector using a different base url can still authenticate.
  • Added missing 429 responses
  • Added support for locking during authentication
  • Added rate limiting for authentication to avoid a 10min lockout if the threshold is exceeded (15 req/5s)
  • Switched php-compatibility version to dev-develop to fix false positive using $this in an enum (there are no recent tagged versions, and apparently none coming for the foreseeable future)
  • Added header parameters missing from generated registration API.

@bob2021 bob2021 changed the title Set type of InvoiceDetails.amount to a string as FedEx require an exact match Improvements to registration and authorization Oct 6, 2025
@bob2021 bob2021 marked this pull request as draft October 6, 2025 19:01
@bob2021 bob2021 marked this pull request as ready for review October 15, 2025 17:54
@colinmollenhour
Copy link
Copy Markdown
Contributor

Awesome work!

@colinmollenhour colinmollenhour merged commit b28c823 into main Oct 17, 2025
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