Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@nnewc
Copy link

@nnewc nnewc commented Dec 11, 2024

Newer versions of Intel AMT use digest auth with different parameters depending on what the server supports. Parsing currently fails when parameters other than "auth" are provided. Running the iamt example I get the following output:

~/src/iamt/example$ ./example 
panic: failed to parse auth header challenge is bad, unexpected token: [realm="Digest:D15CF4BF35F8D0F6CEFD0AA92D4D496A"  nonce="jyTb0E1UAAAAAAAADG/kz8n0/Pb6clW5" stale="false" qop="auth auth-int  auth"]

goroutine 1 [running]:
main.main()
        ~src/iamt/example/main.go:22 +0x272

With my changes, I now get a successful result:

~/src/iamt/example$ ./example 
Is powered on? true

I'm currently using a Minisforum MS-01 with Intel AMT v16 and want to deploy a CAPT cluster.

This PR fixes the parse since, parameters other than "auth" are not supported anyway according to this comment.

@jacobweinstock
Copy link
Owner

Hey @nnewc , thanks for this. Let me do some testing on my end to validate backwards compatibility and then we'll get this merged. Thanks again for this!

@nnewc
Copy link
Author

nnewc commented Dec 12, 2024

Thanks for taking a look. My first Go PR! so totally understand wanting to regression test.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants