whole-tale / girder_wholetale

Girder plugin providing basic Whole Tale functionality
BSD 3-Clause "New" or "Revised" License
3 stars 5 forks source link

Remove unused Globus scope to overcome ORCID 500 #552

Closed craig-willis closed 1 year ago

craig-willis commented 1 year ago

Fixes (for now) https://github.com/whole-tale/girder_wholetale/issues/534

Problem ORCID limits the size of the state param to 2000 characters, otherwise returning 500. This happens on *.local now.

Approach Removed the unused scope to reduce state param size

Test

codecov[bot] commented 1 year ago

Codecov Report

Merging #552 (ae959d2) into master (7a88446) will increase coverage by 0.00%. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #552   +/-   ##
=======================================
  Coverage   92.82%   92.82%           
=======================================
  Files          60       60           
  Lines        4767     4768    +1     
=======================================
+ Hits         4425     4426    +1     
  Misses        342      342           
Impacted Files Coverage Δ
server/__init__.py 90.62% <100.00%> (+0.02%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.