chore: respond with 401 for programmatic requests.

[hamidzr]

to proxied services instead of redirecting those to login page

https://hpe-aiatscale.atlassian.net/browse/GAS-568?filter=-2

Description

Test Plan

after:
hbp ➜ lore git:(main) ✗ curl -H "Accept: application/json" -L http://localhost:8080/genai/favicas -v
*   Trying [::1]:8080...
* Connected to localhost (::1) port 8080
> GET /genai/favicas HTTP/1.1
> Host: localhost:8080
> User-Agent: curl/8.4.0
> Accept: application/json
>
< HTTP/1.1 401 Unauthorized
< Content-Type: application/json; charset=UTF-8
< Vary: Origin
< X-Content-Type-Options: nosniff
< X-Frame-Options: SAMEORIGIN
< X-Xss-Protection: 1; mode=block
< Date: Fri, 02 Feb 2024 23:48:17 GMT
< Content-Length: 27
<
{"message":"Unauthorized"}
* Connection #0 to host localhost left intact

before:
hbp ➜ lore git:(main) ✗ curl -H "Accept: application/json" -L http://localhost:8080/genai/favicas -v
*   Trying [::1]:8080...
* Connected to localhost (::1) port 8080
> GET /genai/favicas HTTP/1.1
> Host: localhost:8080
> User-Agent: curl/8.4.0
> Accept: application/json
>
< HTTP/1.1 303 See Other

Commentary (optional)

Checklist

• Changes have been manually QA'd
• User-facing API changes need the "User-facing API Change" label.
• Release notes should be added as a separate file under docs/release-notes/.
  See Release Note for details.
• Licenses should be included for new code which was copied and/or modified from any external code.

Ticket

[netlify]

✅ Deploy Preview for determined-ui canceled.

Name	Link
🔨 Latest commit	bf1cf26
🔍 Latest deploy log	https://app.netlify.com/sites/determined-ui/deploys/65c17732fa30460008aa8796

[codecov]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (17f305f) 47.72% compared to head (bf1cf26) 47.73%.
Report is 6 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #8795   +/-   ##
=======================================
  Coverage   47.72%   47.73%           
=======================================
  Files        1049     1049           
  Lines      167238   167239    +1     
  Branches     2241     2243    +2     
=======================================
+ Hits        79815    79830   +15     
+ Misses      87265    87251   -14     
  Partials      158      158           

Flag	Coverage Δ
backend	43.28% <85.71%> (+0.04%)	⬆️
harness	64.32% <ø> (ø)
web	42.54% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
master/internal/api_auth.go	44.79% <85.71%> (+16.37%)	⬆️

... and 1 file with indirect coverage changes

[kkunapuli]

LGTM! I left a couple small comments/questions but nothing blocking.

I also understand if you'd prefer to have someone with more context also take a look. :)

[hamidzr]

Thanks for the review. I added some tests to check the behavior as well.