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

[full-ci] add missing cors config #5988

Merged
merged 3 commits into from
Apr 3, 2023
Merged

[full-ci] add missing cors config #5988

merged 3 commits into from
Apr 3, 2023

Conversation

micbar
Copy link
Contributor

@micbar micbar commented Mar 31, 2023

Bugfix: Add missing CORS config

The graph, userlog and ocdav services had no CORS config options.

Reva Part: cs3org/reva#3764

@micbar micbar marked this pull request as draft March 31, 2023 21:30
@micbar micbar force-pushed the add-more-cors branch 2 times, most recently from e9b23ff to 1ced549 Compare April 3, 2023 09:47
@micbar micbar marked this pull request as ready for review April 3, 2023 13:23
@micbar
Copy link
Contributor Author

micbar commented Apr 3, 2023

@mmattel I added defaults and descriptions. Most parts were "just missing"

@micbar
Copy link
Contributor Author

micbar commented Apr 3, 2023

@LukasHirt I think that unblocks the filepicker integration.

Copy link
Contributor

@mmattel mmattel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving, I can only judge that the descriptions are correct but not on the technical content.

Copy link
Contributor

@wkloucek wkloucek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe we should run a full ci run because of updating reva?

go.mod Show resolved Hide resolved
@LukasHirt
Copy link
Collaborator

@LukasHirt I think that unblocks the filepicker integration.

Awesome, thanks! I'll give it a shot once it's ready to confirm it

@micbar micbar changed the title add missing cors config [full-ci] add missing cors config Apr 3, 2023
@ownclouders
Copy link
Contributor

@sonarcloud
Copy link

sonarcloud bot commented Apr 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@micbar micbar merged commit a99058f into master Apr 3, 2023
@delete-merged-branch delete-merged-branch bot deleted the add-more-cors branch April 3, 2023 17:11
ownclouders pushed a commit that referenced this pull request Apr 3, 2023
[full-ci] add missing cors config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants