Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove/Replace the usage of JSONP dataType on AJAX calls when making requests to other ALA services #116

Closed
sughics opened this issue Nov 1, 2022 · 1 comment

Comments

@sughics
Copy link
Contributor

sughics commented Nov 1, 2022

Since JSONP no longer supported i.e. obsolete and is susceptible to known security vulnerabilities, its usage across the app is resulting in CORB(Cross Origin request blocking) .

JSONP usage should be removed/updated where possible to standard JSON.

This is currently causing issues with the new version for biocache-service 3.1.2/3.1.2-SNAPSHOT

Screen Shot 2022-11-01 at 4 14 56 pm

@sughics sughics self-assigned this Nov 1, 2022
sughics added a commit that referenced this issue Nov 1, 2022
…ORB browser error on ajax calls to external ALA services, removing now-redundant .json suffix from biocache-service api endpoints where applicable
sughics added a commit that referenced this issue Nov 2, 2022
issue #116 replacing usage of deprecated jsonp with json to prevent C…
@sughics
Copy link
Contributor Author

sughics commented Nov 16, 2022

Included in 2.4 release

@sughics sughics closed this as completed Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant