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

separate machine auth from system auth api key #3654

Closed
wants to merge 2 commits into from

Conversation

butonic
Copy link
Member

@butonic butonic commented May 3, 2022

We removed the hardcoded MachineAuthAPIKey to make the search service read the key from the config file.

cc @janackermann @aduffeck

@update-docs
Copy link

update-docs bot commented May 3, 2022

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@ownclouders
Copy link
Contributor

ownclouders commented May 3, 2022

💥 Acceptance test cs3ApiTests-ocis failed. Further test are cancelled...

@AlexAndBear
Copy link
Contributor

THX for caring

@butonic butonic force-pushed the remove-secret-in-search-service branch from a479cc1 to 7f58aac Compare May 3, 2022 09:57
@butonic butonic force-pushed the remove-secret-in-search-service branch 4 times, most recently from 49c117f to 86a7913 Compare May 3, 2022 11:43
@butonic butonic changed the title remove leftover hardcoded secret in search service separate machine auth from system auth api key May 3, 2022
@butonic butonic force-pushed the remove-secret-in-search-service branch from 86a7913 to 7a40904 Compare May 3, 2022 12:45
@butonic butonic force-pushed the remove-secret-in-search-service branch from 7a40904 to 783687e Compare May 3, 2022 12:46
@sonarqubecloud
Copy link

sonarqubecloud bot commented May 3, 2022

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 2 Code Smells

0.0% 0.0% Coverage
17.6% 17.6% Duplication

@butonic butonic mentioned this pull request May 4, 2022
45 tasks
@kobergj
Copy link
Collaborator

kobergj commented May 4, 2022

Closed in favor of #3672

@kobergj kobergj closed this May 4, 2022
@butonic butonic deleted the remove-secret-in-search-service branch November 7, 2022 08:39
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.

4 participants