Skip to content

Authentication Tests Update #3038

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

Merged
merged 1 commit into from
May 24, 2025
Merged

Authentication Tests Update #3038

merged 1 commit into from
May 24, 2025

Conversation

zapbot
Copy link
Contributor

@zapbot zapbot commented Apr 1, 2025

Signed-off-by: zapbot [email protected]

@psiinon
Copy link
Member

psiinon commented Apr 1, 2025

Logo
Checkmarx One – Scan Summary & Detailsfa85412f-7509-4e80-bba4-e5be83cef510

Great job, no security vulnerabilities found in this Pull Request

@zapbot zapbot force-pushed the auth-tests-update branch 8 times, most recently from d7af577 to b768492 Compare April 9, 2025 03:57
@zapbot zapbot force-pushed the auth-tests-update branch 7 times, most recently from f711dc2 to 51edb07 Compare April 16, 2025 03:59
@zapbot zapbot force-pushed the auth-tests-update branch 7 times, most recently from 6efdbfb to 865e83e Compare April 23, 2025 03:59
@zapbot zapbot force-pushed the auth-tests-update branch 7 times, most recently from 4c2f7fa to ca173e1 Compare April 30, 2025 04:00
@zapbot zapbot force-pushed the auth-tests-update branch 3 times, most recently from bd7e4b6 to 0958fe2 Compare May 15, 2025 12:29
@kingthorin
Copy link
Member

I think we may as well merge this. It removes the authmulti bbaplus which we disabled.

@zapbot zapbot force-pushed the auth-tests-update branch 7 times, most recently from 5dcbc7e to a42d5fc Compare May 22, 2025 04:04
@zapbot zapbot force-pushed the auth-tests-update branch from a42d5fc to 57b4c99 Compare May 23, 2025 04:04
@thc202
Copy link
Member

thc202 commented May 23, 2025

Why is authtestmulti bbaplus still changing? (It's also removing stdbba not bbaplus.)

@kingthorin
Copy link
Member

Hmmm that's a good question, it should be being skipped entirely.

I'll have a look.

@kingthorin
Copy link
Member

@thc202
Copy link
Member

thc202 commented May 23, 2025

That disables everything, which didn't seem to be the objective (I might got things wrong though).

@zapbot zapbot force-pushed the auth-tests-update branch from 57b4c99 to 2971e11 Compare May 23, 2025 10:48
@kingthorin
Copy link
Member

Actually apparently I only disabled standard. Will adjust.

@zapbot zapbot force-pushed the auth-tests-update branch from 2971e11 to cc99cba Compare May 23, 2025 13:33
@zapbot zapbot force-pushed the auth-tests-update branch from cc99cba to 46fa133 Compare May 24, 2025 03:59
@psiinon psiinon merged commit d8f0d4c into zaproxy:main May 24, 2025
3 checks passed
@thc202
Copy link
Member

thc202 commented May 24, 2025

Tomorrow we are going to have this PR again…

@thc202
Copy link
Member

thc202 commented May 25, 2025

We didn't, maybe a lucky day 🍀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants