Skip to content

Improve jwt logging on failed auth #122247

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
jfreden merged 1 commit into from
Feb 17, 2025
Merged

Improve jwt logging on failed auth #122247

jfreden merged 1 commit into from
Feb 17, 2025

Conversation

@jfreden
Copy link
Contributor

@jfreden jfreden commented Feb 11, 2025 •
edited
Loading

Improve jwt logging on failed auth

@jfreden jfreden added the :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) label Feb 11, 2025
@elasticsearchmachine elasticsearchmachine added Team:Security Meta label for security team v9.1.0 labels Feb 11, 2025
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Feb 11, 2025

Pinging @elastic/es-security (Team:Security)

@jfreden jfreden added the >bug label Feb 11, 2025
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Feb 11, 2025

Hi @jfreden, I've created a changelog YAML for you.

@jfreden jfreden force-pushed the fix_stack_overflow_error branch 2 times, most recently from fadedd7 to 3a233d5 Compare February 11, 2025 14:24
@jfreden jfreden changed the title Large nested claim set causes stack overflow in toString Improve jwt logging on failed auth Feb 11, 2025
@jfreden jfreden force-pushed the fix_stack_overflow_error branch from 3a233d5 to c6a936e Compare February 11, 2025 15:40
@jfreden
Improve jwt logging on failed auth
bc2ab83 
Update docs/changelog/122247.yaml
@jfreden jfreden force-pushed the fix_stack_overflow_error branch from c6a936e to bc2ab83 Compare February 11, 2025 15:42
@jfreden jfreden added v8.18.1 v8.19.0 v9.0.1 v8.16.5 v8.17.3 auto-backport Automatically create backport pull requests when merged labels Feb 17, 2025
slobodanadamovic
slobodanadamovic approved these changes Feb 17, 2025
View reviewed changes
Copy link
Contributor

@slobodanadamovic slobodanadamovic left a comment

Choose a reason for hiding this comment

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

LGTM

@jfreden jfreden merged commit 1f4f5dd into elastic:main Feb 17, 2025
22 checks passed
This was referenced Feb 17, 2025
Merged
Merged
Merged
jfreden added a commit to jfreden/elasticsearch that referenced this pull request Feb 17, 2025
@jfreden
Improve jwt logging on failed auth (elastic#122247)
e074ca8 
Update docs/changelog/122247.yaml
@jfreden jfreden mentioned this pull request Feb 17, 2025
Merged
jfreden added a commit to jfreden/elasticsearch that referenced this pull request Feb 17, 2025
@jfreden
Improve jwt logging on failed auth (elastic#122247)
da6ee16 
Update docs/changelog/122247.yaml
@jfreden jfreden mentioned this pull request Feb 17, 2025
Merged
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Feb 17, 2025

💚 Backport successful

Status Branch Result
✅ 8.18
✅ 8.x
✅ 9.0
✅ 8.16
✅ 8.17

jfreden added a commit to jfreden/elasticsearch that referenced this pull request Feb 17, 2025
@jfreden
Improve jwt logging on failed auth (elastic#122247)
f72f52c 
Update docs/changelog/122247.yaml
jfreden added a commit to jfreden/elasticsearch that referenced this pull request Feb 17, 2025
@jfreden
Improve jwt logging on failed auth (elastic#122247)
a546d5a 
Update docs/changelog/122247.yaml
jfreden added a commit to jfreden/elasticsearch that referenced this pull request Feb 17, 2025
@jfreden
Improve jwt logging on failed auth (elastic#122247)
f816337 
Update docs/changelog/122247.yaml
elasticsearchmachine pushed a commit that referenced this pull request Feb 17, 2025
@jfreden
Improve jwt logging on failed auth (#122247) (#122784)
4f9c33f 
Update docs/changelog/122247.yaml
elasticsearchmachine pushed a commit that referenced this pull request Feb 17, 2025
@jfreden
Improve jwt logging on failed auth (#122247) (#122785)
61aec94 
Update docs/changelog/122247.yaml
elasticsearchmachine pushed a commit that referenced this pull request Feb 21, 2025
@jfreden
Improve jwt logging on failed auth (#122247) (#122783)
63d0b6a 
Update docs/changelog/122247.yaml
elasticsearchmachine pushed a commit that referenced this pull request Feb 21, 2025
@jfreden
Improve jwt logging on failed auth (#122247) (#122782)
230fb30 
Update docs/changelog/122247.yaml
elasticsearchmachine pushed a commit that referenced this pull request Feb 21, 2025
@jfreden
Improve jwt logging on failed auth (#122247) (#122781)
2f0951e 
Update docs/changelog/122247.yaml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@slobodanadamovic slobodanadamovic slobodanadamovic approved these changes

Assignees

No one assigned

Labels

auto-backport Automatically create backport pull requests when merged >bug :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) Team:Security Meta label for security team v8.16.5 v8.17.3 v8.18.1 v8.19.0 v9.0.1 v9.1.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jfreden @elasticsearchmachine @slobodanadamovic