[Patch] pki-vakwetu-0006-BZ-771357-sslget-does-not-work-after-FEDORA-2011-174
by Ade Lee
Updated patch to concatenate the first 13 bytes correctly. Please
review.
Tested using latest NSS and f15 using RA. Requests now succeed.
Also tested using latest NSS and f16 using IPA install. IPA install now
successful.
Note:
The check in the current code is not really valid. Right now, it
checks the first 13 bytes - which should contain the following:
HTTP/1.1 200 OK
It then checks if 200 is a substring of the first 13 bytes - and exits
with a failure otherwise. This isn't the right thing to do. For one
thing, valid status codes like 201, 202 etc. would result in failure.
This will become a problem when interacting with the new restful
interface which will return codes other than 200 OK.
I will revisit and fix this in a separate bug.
Ade
13 years, 1 month
[Patch] pki-vakwetu-0005-BZ-771357-sslget-does-not-work-after-FEDORA-2011-174
by Ade Lee
Please review. I tested using an RA. When I updated NSS, user
enrollments failed with a failure to parse the sslget response. When I
added my patch, everything succeeded once again. I tested on F15.
Matt, if approved, please apply to all the relevant branches. Also, as
a final test - perhaps ask Alexander (irc ab) to test using an IPA
install to confirm that it works for him.
Thanks,
Ade
13 years, 1 month