Skip to content

Update Getting Started/Events for Additional Secrets#779

Merged
sven-bitwarden merged 3 commits into
mainfrom
update-events-getting-started
Mar 12, 2026
Merged

Update Getting Started/Events for Additional Secrets#779
sven-bitwarden merged 3 commits into
mainfrom
update-events-getting-started

Conversation

@sven-bitwarden
Copy link
Copy Markdown
Contributor

@sven-bitwarden sven-bitwarden commented Mar 4, 2026

📔 Objective

When spinning up the EventsProcessor using Azure Service Bus, it injects an IEventRepository backed by TableStorageRepos.EventRepository (source).

This expects globalSettings.Events.ConnectionString to be set, otherwise an exception is thrown.

@sven-bitwarden sven-bitwarden requested a review from a team as a code owner March 4, 2026 19:35
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 4, 2026

Logo
Checkmarx One – Scan Summary & Details2585cc9e-00f2-43dc-8f28-1f494d68e677

Great job! No new security vulnerabilities introduced in this pull request

Copy link
Copy Markdown
Member

@eliykat eliykat left a comment

Choose a reason for hiding this comment

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

Good find.

Comment thread docs/getting-started/server/events.md Outdated
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Mar 11, 2026

Deploying contributing-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 985a529
Status: ✅  Deploy successful!
Preview URL: https://f8f91707.contributing-docs.pages.dev
Branch Preview URL: https://update-events-getting-starte.contributing-docs.pages.dev

View logs

@sven-bitwarden sven-bitwarden requested a review from eliykat March 11, 2026 18:34
@sven-bitwarden sven-bitwarden merged commit 03a079f into main Mar 12, 2026
13 checks passed
@sven-bitwarden sven-bitwarden deleted the update-events-getting-started branch March 12, 2026 17:19
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