Skip to content

Bug-475: metadig-controller CORS Access-Control-Allow-Origin Issue #476

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.

Sign up for GitHub

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

Jump to bottom
Merged
doulikecookiedough merged 15 commits into from
Apr 1, 2025
Merged

Bug-475: metadig-controller CORS Access-Control-Allow-Origin Issue #476

doulikecookiedough merged 15 commits into from
Apr 1, 2025

Conversation

@doulikecookiedough
Copy link
Contributor

@doulikecookiedough doulikecookiedough commented Apr 1, 2025

Summary of Changes:

  • Updated metadig-controller's ingress.yaml and values.yaml with bespoke CORS setting to resolve No 'Access-Control-Allow-Origin' issue
  • Minor additions for logging to assist with debugging

@doulikecookiedough
Add debug statement (printStackTrace) to 'Worker' where jep exception…
5fa4b74 
… is encountered
@doulikecookiedough
Revert code in 'MDQEngine' class temporarily when checking for null n…
769cd36 
…odeList for debugging purposes
@doulikecookiedough
Revise 'DataONE' class log messaging format to be easier to view
6bfa2e6
@doulikecookiedough
Change trace level log statement to debug level in 'DataONE' class fo…
25d817a 
…r making a solr query
@doulikecookiedough
Add new debug statement to 'MDQEngine' class to confirm solr query
37abd51
@doulikecookiedough
Add clarity to 'DataONE' clas debug statement for solr query
910afa4
@doulikecookiedough
Add more debug statements to 'RequestReportJob' class
0fbae25
@doulikecookiedough
Code check-in: Update metadig-controller 'values.yaml' and 'ingress.y…
a967ea0 
…aml' to resolve 'Access-Control-Allow-Origin' issue
@doulikecookiedough
Code check-in: metadig-controller ingress cors debugging
f799b81
@doulikecookiedough
Change debug levels back to trace in 'RequestReportJob' class
07bd70b
@doulikecookiedough
Re-activate guard rails for missing nodeList in 'findDataPids' method…
6dda1f0 
… in 'MDQEngine' class
@doulikecookiedough
Clean up comments in 'MDQEngine' and 'Worker' class
99c6eb6
@doulikecookiedough
Revert host path values in values.yaml for 'metadig-controller'
7188aee
@doulikecookiedough
Revert host & tls values in 'values.yaml' for 'metadig-controller'
c2077ee
@doulikecookiedough
Update values.yaml for 'metadig-controller' to have hosts for prod an…
0ade29c 
…d dev
@doulikecookiedough doulikecookiedough merged commit 5383b34 into develop Apr 1, 2025
2 checks passed
@doulikecookiedough doulikecookiedough mentioned this pull request Apr 1, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@doulikecookiedough