bbockelm commented on this pull request.
> + if (eq == std::string::npos)
+ {
+ return req.SendSimpleResp(400, NULL, NULL, "Invalid format for form-encoding", 0);
+ }
+ std::string key = token.substr(0, eq);
+ std::string value = token.substr(eq + 1);
+ //std::cout << "Found key " << key << ", value " << value << std::endl;
+ if (key == "grant_type")
+ {
+ found_grant_type = true;
+ if (value != "client_credentials")
+ {
+ return req.SendSimpleResp(400, NULL, NULL, "Invalid grant type specified.", 0);
+ }
+ }
+ else if (key == "expire_in")
Yes - this was a bit arbitrary on my part. I didn't know of an "obvious" candidate here.
Any reason that `expires_in` versus `expire_in` versus `lifetime` might be preferred? Shall we start flipping coins?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/xrootd/xrootd/pull/883#discussion_r242989272
########################################################################
Use REPLY-ALL to reply to list
To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1
