Skip to content

fix: (secretmanaer) readd legacy path templates#5290

Merged
sofisl merged 58 commits intomainfrom
secretManagerFixes
May 2, 2024
Merged

fix: (secretmanaer) readd legacy path templates#5290
sofisl merged 58 commits intomainfrom
secretManagerFixes

Conversation

@sofisl
Copy link
Contributor

@sofisl sofisl commented Apr 29, 2024

Fixes #5289

@sofisl sofisl requested review from a team and yoshi-approver as code owners April 29, 2024 20:13
@sofisl sofisl changed the title fix: readd legacy path templates fix: (secretmanaer) readd legacy path templates Apr 30, 2024
sofisl and others added 23 commits April 29, 2024 22:38
gcf-owl-bot bot and others added 17 commits April 30, 2024 03:53
@sofisl sofisl added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 30, 2024
@sofisl sofisl force-pushed the secretManagerFixes branch from f31f642 to e16436e Compare May 1, 2024 19:34
@sofisl sofisl removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label May 1, 2024
@sofisl sofisl merged commit 5e14553 into main May 2, 2024
@sofisl sofisl deleted the secretManagerFixes branch May 2, 2024 16:36
@release-please release-please bot mentioned this pull request May 2, 2024
@flovouin
Copy link

flovouin commented May 17, 2024

This does not seem to work:

TypeError: Cannot read properties of undefined (reading 'render')
    at SecretManagerServiceClient.secretVersionPath (node_modules/@google-cloud/secret-manager/build/src/v1/secret_manager_service_client.js:1161:61)

And indeed, secretVersionPathTemplate does not exist in pathTemplates.

EDIT: Saw that this could be fixed by #5333.

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.

Breaking change in minor release of @google-cloud/secret-manager 5.4.0

3 participants