Skip to content

Enable redirect for httplib#437

Closed
urfeex wants to merge 1 commit intoUniversalRobots:masterfrom
urfeex:add_redirect
Closed

Enable redirect for httplib#437
urfeex wants to merge 1 commit intoUniversalRobots:masterfrom
urfeex:add_redirect

Conversation

@urfeex
Copy link
Member

@urfeex urfeex commented Feb 13, 2026

Some targets have changed between versions redirecting to the correct endpoint. For this to work, we'll have to follow redirects, which is not the default for httplib.

Resolves #436

Some targets have changed between versions redirecting to the correct
endpoint. For this to work, we'll have to follow redirects, which is not
the default for httplib.
@urfeex urfeex requested a review from a team February 13, 2026 08:19
@urfeex urfeex added the bugfix label Feb 13, 2026
@codecov
Copy link

codecov bot commented Feb 13, 2026

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 74.93%. Comparing base (08c33c3) to head (6305874).

Files with missing lines Patch % Lines
src/ur/dashboard_client_implementation_x.cpp 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #437      +/-   ##
==========================================
- Coverage   75.33%   74.93%   -0.40%     
==========================================
  Files         102      102              
  Lines        5120     5119       -1     
  Branches      544      543       -1     
==========================================
- Hits         3857     3836      -21     
- Misses        980      998      +18     
- Partials      283      285       +2     
Flag Coverage Δ
start_ursim 82.72% <ø> (-1.90%) ⬇️
ur5-3.14.3 73.13% <0.00%> (-0.17%) ⬇️
ur5e-10.7.0 65.07% <0.00%> (-0.24%) ⬇️
ur5e-5.9.4 73.40% <0.00%> (-0.12%) ⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@urfeex
Copy link
Member Author

urfeex commented Feb 13, 2026

Closing this in favor of #435 which also includes this.

@urfeex urfeex closed this Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Polyscope X Dashboard doesn't follow redirects

1 participant