Skip to content

feat: one-password integration - #164

Merged
arpit-bruno merged 2 commits into
opencollection-dev:mainfrom
prateek-bruno:feat/add-one-password-secret-manager
Oct 28, 2026
Merged

arpit-bruno merged 2 commits into
opencollection-dev:mainfrom
prateek-bruno:feat/add-one-password-secret-manager

Conversation

@prateek-bruno

@prateek-bruno prateek-bruno commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor

Internal: BRU-4809

Api Docs: usebruno/bruno-api-docs#107

Adds one-password as an external secret provider. Each variable takes a secretReference: the op:// reference to the item field that holds the value.

@sanish-bruno

Copy link
Copy Markdown

@prateek-bruno we need to update here too https://github.com/usebruno/bruno-api-docs

@sanish-bruno sanish-bruno 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.

@sreelakshmi-bruno Please take a look once you have time, Let us know if we need to update it

},
"secretReference": {
"type": "string",
"description": "The 1Password secret reference to the item field that holds the value"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

can consider updating the description A 1Password secret reference (op://) identifying the field containing the secret value,
The 1Password secret reference used to retrieve the secret value,
The 1Password secret reference (op://) used to resolve the secret value

@prateek-bruno prateek-bruno Oct 19, 2026 •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I am fine with this:
The 1Password secret reference used to retrieve the secret value,

But for consistency we shouldn't mention op://

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I have updated it to this :
The secret reference to the field within 1Password that holds the value

@arpit-bruno
arpit-bruno merged commit 8745d22 into opencollection-dev:main Oct 28, 2026
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