Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[PR #7241/1beb38ce backport][stable-7] Use get(..) instead of [..] for safe lookup of value (Fixes #7240) #7258

Merged
merged 1 commit into from
Sep 13, 2023
Merged

[PR #7241/1beb38ce backport][stable-7] Use get(..) instead of [..] for safe lookup of value (Fixes #7240) #7258

merged 1 commit into from
Sep 13, 2023

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Sep 13, 2023

This is a backport of PR #7241 as merged into main (1beb38c).

A OnePassword field item might not have a value (property) when the user has omitted it (on purpose).

SUMMARY
ISSUE TYPE
COMPONENT NAME

OnePassword

ADDITIONAL INFORMATION

@wkleinheerenbrink @patchback
Use get(..) instead of [..] for safe lookup of value (Fixes #7240) (#…
8217408 
…7241)

A OnePassword field item might not have a value (property) when the user has omitted it (on purpose).

(cherry picked from commit 1beb38c)
@patchback patchback bot mentioned this pull request Sep 13, 2023
Merged
@ansibullbot
Copy link
Collaborator

cc @azenk @samdoran
click here for bot help

@ansibullbot ansibullbot added backport bug This issue/PR relates to a bug lookup lookup plugin new_contributor Help guide this first time contributor plugins plugin (any type) tests tests unit tests/unit labels Sep 13, 2023
@felixfontein felixfontein merged commit aa3b53f into stable-7 Sep 13, 2023
@felixfontein felixfontein deleted the patchback/backports/stable-7/1beb38ceff80b9e1d1437cab2ba27deff6a73124/pr-7241 branch September 13, 2023 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug This issue/PR relates to a bug lookup lookup plugin new_contributor Help guide this first time contributor plugins plugin (any type) tests tests unit tests/unit
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ansibullbot @felixfontein @wkleinheerenbrink