search

broadinstitute / openidc-terra-proxy

OpenIDC proxy to be used for all Terra services
BSD 3-Clause "New" or "Revised" License
2 stars 0 forks source link

Disable Expect: 100-continue in the php introspection endpoint #5

Closed rtitle closed 2 years ago

rtitle commented 2 years ago

While re-testing this, I see an added ~1s latency in the POST request to https://www.googleapis.com/oauth2/v3/tokeninfo. Seeing if this fixes it (related to https://docs.google.com/document/d/1ZV-meiMt2F3tsgifXCaTrGBLFTyBN1tNKU-nFwSk-lE/edit).

github-actions[bot] commented 2 years ago

Pushed image: us.gcr.io/broad-dsp-gcr-public/openidc-terra-proxy:v0.1.5-a2466b3

github-actions[bot] commented 2 years ago

Pushed image: us.gcr.io/broad-dsp-gcr-public/openidc-terra-proxy:v0.1.5-b858a4b

rtitle commented 2 years ago

It seems to help.

Before:

127.0.0.1 - - "-" "-" [2022-03-01T10:08:59.590-0500] 1037926 - "POST /introspect/ HTTP/1.1" 200 359 "-" "liboauth2-1.4.2"

1037926us == 1.038s

After:

127.0.0.1 - - "-" "-" [2022-03-01T10:36:02.975-0500] 47895 - "POST /introspect/ HTTP/1.1" 200 439 "-" "liboauth2-1.4.2"

47895us == 48ms

rtitle commented 2 years ago

@nmalfroy this is the fix I mentioned earlier