Skip to content

Conversation

@tobio
Copy link
Member

@tobio tobio commented Jan 11, 2025

Fixes #956

@Leaf-Lin is this the change you're after (you can just look at the first commit)? Otherwise, want to let me know what you'd like this to say?

@tobio tobio requested review from Leaf-Lin and dimuon January 11, 2025 05:59
@tobio tobio self-assigned this Jan 11, 2025
Copy link

@Leaf-Lin Leaf-Lin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the quick fix! I didn't realise it has been mentioned in so many different places. This will clear up the usage of the variable.

@tobio
Copy link
Member Author

tobio commented Jan 14, 2025

Thank you for the quick fix! I didn't realise it has been mentioned in so many different places. This will clear up the usage of the variable.

All good, the docs are generated so it's only one code change.

@tobio tobio merged commit 08b9042 into main Jan 14, 2025
45 checks passed
@tobio tobio deleted the client-auth-docs-fix branch January 14, 2025 06:32
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.

[Bug] security es_client_authentication should refer to the JWT token

3 participants