Skip to content

feat(upload): Use dedicated secret to sign location URLs#6132

Open
jjbayer wants to merge 20 commits into
masterfrom
feat/upload-secrets
Open

feat(upload): Use dedicated secret to sign location URLs#6132
jjbayer wants to merge 20 commits into
masterfrom
feat/upload-secrets

Conversation

@jjbayer

@jjbayer jjbayer commented Jun 24, 2026

Copy link
Copy Markdown
Member

Allow to configure a separate key pair for signing and verifying upload signatures. This makes signature logic independent of the relay's identity, and will allow us to for example add verification to pop relays (if they share the same key pair as processing relays).

ref: INGEST-755

@linear-code

linear-code Bot commented Jun 24, 2026

Copy link
Copy Markdown

INGEST-755

@jjbayer jjbayer marked this pull request as ready for review June 25, 2026 11:50
@jjbayer jjbayer requested a review from a team as a code owner June 25, 2026 11:50
Comment thread relay-config/src/config.rs
Comment thread relay-server/src/services/upload.rs
Comment thread relay-config/src/config.rs
Comment thread relay-config/src/config.rs Outdated
Comment thread relay-config/src/config.rs Outdated
Comment thread relay-server/src/services/upload.rs
Comment thread relay-server/src/services/upload.rs Outdated

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit f524e56. Configure here.

Comment thread relay-server/src/services/upload.rs
Comment thread relay-config/src/config.rs
Comment thread relay-server/src/services/upload.rs
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.

2 participants