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

Issue With Hash/Supplied Password #68

Open
DavidBittner opened this issue Nov 15, 2019 · 3 comments
Open

Issue With Hash/Supplied Password #68

DavidBittner opened this issue Nov 15, 2019 · 3 comments
Labels
Question Need some information Resolved The issue is resolved or got answered

Comments

@DavidBittner
Copy link
Contributor

Hello,

There appears to be an issue in the cs531a4: test_correct_realm_authorized. No matter what I changed, I could not seem to get the right hash. I decided to look at the source code of the tester itself that you have on your GitHub, and looking at line 21, a1 is generated using the string "mln:Colonial Place:mln".

Assuming that last field there, mln is supposed to be the password, the password supplied in our WeMustProtectThisHouse! file says the password is 53bbb5135e0f39c1eb54804a66a95f08. If I changed my a1 to hardcode the string value that I found in the source of the HttpTester it works fine.

Is this just a small mistake? Or am I misunderstanding how we grab the password?

Thanks a ton,
David

@ibnesayeed ibnesayeed added the Question Need some information label Nov 15, 2019
@ibnesayeed
Copy link
Collaborator

Unlike the basic auth, in digest auth we store hash of <username>:<realm>:<password> in the config file not the has of the password. We discussed last week's lecture while revisiting assignment 4 and Auth lecture slides.

$ echo -n "mln" | md5sum
ae33d20c70e59a4c734d9f2c19c0df56  -

$ echo -n "mln:Colonial Place:mln" | md5sum
53bbb5135e0f39c1eb54804a66a95f08  -

@DavidBittner
Copy link
Contributor Author

Thanks! Good example of something I would've picked up in class. I'll make sure to be there next Thursday. Have a great weekend!

@ibnesayeed
Copy link
Collaborator

Since we are using this as a discussion board and not bug tracker, I would keep it open, but add a the Resolved label to it.

@ibnesayeed ibnesayeed reopened this Nov 15, 2019
@ibnesayeed ibnesayeed added the Resolved The issue is resolved or got answered label Nov 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Question Need some information Resolved The issue is resolved or got answered
Projects
None yet
Development

No branches or pull requests

2 participants