Skip to content

feat: job decrypt#123

Merged
vikaxsh merged 1 commit into
stagingfrom
feat/job-decrypt
May 19, 2026
Merged

feat: job decrypt#123
vikaxsh merged 1 commit into
stagingfrom
feat/job-decrypt

Conversation

@schitizsharma
Copy link
Copy Markdown
Collaborator

Description

This PR adds decryption support to the olake-worker to ensure that encrypted source and destination configurations can be read from the database. The implementation mirrors the server-side decryption logic, allowing for seamless communication between the UI/Server and the Worker.

Key Changes:

  • New Utility: Created worker/utils/encryption.go to handle decryption using a key derived from the OLAKE_SECRET_KEY environment variable.
  • Support for Multiple Modes: Implement both Local AES-256-GCM and AWS KMS decryption modes (triggered if the secret key is an ARN).
  • Core Update: Updated worker/database/job.go to automatically decrypt Source and Destination configs during GetJobData.
  • Dependencies: Added AWS SDK v2 dependencies to worker/go.mod.

Fixes # (issue)

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • Build Verification: Ran go build ./... in the worker directory to confirm all dependencies are correctly resolved.
  • Code Review: Verified that the AES-GCM and KMS decryption flow matches the implementation in olake-ui/server/utils/encryption.go.

Screenshots or Recordings

N/A (Backend logic change)

Related PR's (If Any):

N/A

@schitizsharma schitizsharma changed the title Feat/job decrypt feat: job decrypt Apr 2, 2026
@schitizsharma schitizsharma marked this pull request as ready for review May 18, 2026 07:43
Decrypt AES-256-GCM (local) and AWS KMS encrypted configs
stored in the database before passing them to job execution.
If OLAKE_SECRET_KEY is not set, configs are used as-is.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vikaxsh vikaxsh force-pushed the feat/job-decrypt branch from 9da50fe to 3542c69 Compare May 19, 2026 10:59
@vikaxsh vikaxsh merged commit 46d9b05 into staging May 19, 2026
5 checks passed
@vikaxsh vikaxsh deleted the feat/job-decrypt branch May 19, 2026 16:49
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