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

accounts now uses the same reva jwt secret as the metadata storage #3081

Merged
merged 1 commit into from
Feb 1, 2022

Conversation

wkloucek
Copy link
Contributor

@wkloucek wkloucek commented Feb 1, 2022

Description

Bugfix: use same jwt secret for accounts as for metadata storage

We've the metadata storage uses the same jwt secret as all other REVA services.
Therefore the accounts service needs to use the same secret.

Secrets are documented here: https://owncloud.dev/ocis/deployment/#change-default-secrets

Motivation and Context

Start oCIS with following environment variables:

OCIS_JWT_SECRET: "random-756"
STORAGE_TRANSFER_SECRET: "random-987"
OCIS_MACHINE_AUTH_API_KEY: "random-123"

and the accounts service can't write / read from the metadata storage...

@update-docs
Copy link

update-docs bot commented Feb 1, 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.

@wkloucek wkloucek requested review from butonic, C0rby, micbar and kobergj and removed request for kulmann and pascalwengerter February 1, 2022 14:29
@sonarcloud
Copy link

sonarcloud bot commented Feb 1, 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 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@wkloucek wkloucek merged commit 3eeffaa into master Feb 1, 2022
@delete-merged-branch delete-merged-branch bot deleted the metadata-service-jwt-token branch February 1, 2022 15:13
ownclouders pushed a commit that referenced this pull request Feb 1, 2022
Merge: 3cbd78f f0d072e
Author: Willy Kloucek <34452982+wkloucek@users.noreply.github.com>
Date:   Tue Feb 1 16:13:23 2022 +0100

    Merge pull request #3081 from owncloud/metadata-service-jwt-token

    accounts now uses the same reva jwt secret as the metadata storage
@micbar micbar mentioned this pull request Feb 16, 2022
22 tasks
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.

3 participants