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.

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

Added partial vulnerability in alibaba nacos application. Default password and unauthorized access bypass #56

Merged
merged 15 commits into from
Jul 18, 2023
Merged

Added partial vulnerability in alibaba nacos application. Default password and unauthorized access bypass #56

merged 15 commits into from
Jul 18, 2023

Conversation

JaveleyQAQ
Copy link
Contributor

No description provided.

PortSwiggerWiener
PortSwiggerWiener reviewed Jul 17, 2023
View reviewed changes
Copy link
Collaborator

@PortSwiggerWiener PortSwiggerWiener left a comment

Choose a reason for hiding this comment

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

Looks interesting. Please can be make the indentation consistent.

@JaveleyQAQ
Copy link
Contributor Author

Looks interesting. Please can be make the indentation consistent.看起来很有趣请能使压痕一致。

No problem. I forgot to change the testing part from "given request then" to "given host then" 😭 now already been changed

PortSwiggerWiener
PortSwiggerWiener approved these changes Jul 17, 2023
View reviewed changes
Copy link
Collaborator

@PortSwiggerWiener PortSwiggerWiener left a comment

Choose a reason for hiding this comment

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

Thanks for making the changes. Looks good 👍

@olliewuk olliewuk merged commit dd86b93 into PortSwigger:main Jul 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@olliewuk olliewuk olliewuk approved these changes

@PortSwiggerWiener PortSwiggerWiener PortSwiggerWiener approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@JaveleyQAQ @olliewuk @PortSwiggerWiener