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

Fix 500 error in content app when passed bad cookies #6255

Open

gerrod3 wants to merge 1 commit into pulp:main from gerrod3:content-bad-cookie

+25 −2

Contributor

gerrod3 commented Feb 3, 2025

fixes: #6214

Another one solved by the Cursor agent, the commands I issued:

Fix the issue described here: @https://github.com/pulp/pulpcore/issues/6214 

The exception catched should be CookieError

Rework the exception to return a 400 with the message 'Received invalid cookies'

Let's add a functional test 

Add a changelog for the issue


          Fix 500 error in content app when passed bad cookies

b832745

fixes: pulp#6214

gerrod3 force-pushed the content-bad-cookie branch from 3a5c27e to b832745 Compare

February 3, 2025 21:18

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

Labels

None yet