Skip to content

Fix token auth prefix removal.#721

Merged
markstory merged 3 commits into3.xfrom
bugfix/token-auth
May 9, 2025
Merged

Fix token auth prefix removal.#721
markstory merged 3 commits into3.xfrom
bugfix/token-auth

Conversation

@dereuromark
Copy link
Copy Markdown
Member

Resolves #720

Does this work?

However, the test case doesnt quite seem right, it fails.
Do you have an idea for a correct test case?

@dereuromark dereuromark added the bug label May 2, 2025
@dereuromark dereuromark added this to the 3.x milestone May 2, 2025
@dereuromark dereuromark marked this pull request as ready for review May 2, 2025 13:06
@dereuromark dereuromark requested a review from ADmad May 2, 2025 13:11
dereuromark and others added 2 commits May 7, 2025 19:53
@markstory markstory merged commit 39e565c into 3.x May 9, 2025
8 checks passed
@markstory markstory deleted the bugfix/token-auth branch May 9, 2025 03:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

str_ireplace replaces $prefix if $prefix is embedded in token string now

2 participants