Skip to content
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.

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

Added the string corresponding to "{"alg":" since some token headers start with this #5012

Merged
merged 3 commits into from
Oct 6, 2024

Conversation

ionsor
Copy link
Contributor

@ionsor ionsor commented Sep 18, 2024

Summary of the Pull Request

Added the string eyJhbGciOi corresponding to {"alg": from the JWT token header.

Changelog

update: Potentially Suspicious JWT Token Search Via CLI - added the eyJhbGciOi string, corresponding to {"alg": from the JWT token header.

Example Log Event

image

Fixed Issues

N/A

SigmaHQ Rule Creation Conventions

  • If your PR adds new rules, please consider following and applying these conventions

@github-actions github-actions bot added Rules Windows Pull request add/update windows related rules labels Sep 18, 2024
Copy link
Contributor Author

@ionsor ionsor left a comment

Choose a reason for hiding this comment

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

From my hunts I observed JWT tokens starting with "typ" either "alg", hence added the corresponding base64 string missing from the detection

@nasbench nasbench added the Work In Progress Some changes are needed label Sep 18, 2024
@nasbench nasbench self-assigned this Sep 18, 2024
@nasbench nasbench self-requested a review September 18, 2024 21:56
@nasbench
Copy link
Member

From my hunts I observed JWT tokens starting with "typ" either "alg", hence added the corresponding base64 string missing from the detection

For reviewing and testing purposes, can you provide a log sample. As you seem to have one at hand already. Thanks

@ionsor
Copy link
Contributor Author

ionsor commented Oct 1, 2024

From my hunts I observed JWT tokens starting with "typ" either "alg", hence added the corresponding base64 string missing from the detection

For reviewing and testing purposes, can you provide a log sample. As you seem to have one at hand already. Thanks

Sure. Please see the attached printscreen from Chrome developer tools, hope it helps, otherwise the logs are collected through an EDR.
image

@nasbench nasbench removed the Work In Progress Some changes are needed label Oct 6, 2024
@nasbench nasbench added 2nd Review Needed PR need a second approval and removed 2nd Review Needed PR need a second approval labels Oct 6, 2024
@nasbench nasbench merged commit 5b59c6d into SigmaHQ:master Oct 6, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Rules Windows Pull request add/update windows related rules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants