FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

ROX-18155: pg generic store: Get by janisz · Pull Request #6770 · stackrox/stackrox · GitHub

/ stackrox Public

ROX-18155: pg generic store: Get - #6770

Merged
janisz merged 1 commit into
masterfrom
master-janisz/06-30-ROX-18155_pg_generic_store_Get
Jul 7, 2023
ConversationCommits (1)ChecksFiles changed
Merged

ROX-18155: pg generic store: Get#6770
janisz merged 1 commit into
masterfrom
master-janisz/06-30-ROX-18155_pg_generic_store_Get

Conversation

janisz commented Jun 30, 2023

Copy link
Copy Markdown
Contributor

Description

A detailed explanation of the changes in your PR.

Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.

Checklist

  • Investigated and inspected CI test results
  • Unit test and regression tests added
  • Evaluated and added CHANGELOG entry if required
  • Determined and documented upgrade steps
  • Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.

Testing Performed

TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.

In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

janisz requested review from a team as code owners June 30, 2023 16:19
janisz mentioned this pull request Jun 30, 2023
Merged
5 tasks
This was referenced Jun 30, 2023
Merged
Merged
Merged

janisz commented Jun 30, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

This was referenced Jun 30, 2023
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged

ghost commented Jun 30, 2023 •
edited by ghost
Loading

Copy link
Copy Markdown

Images are ready for the commit at 9bdc2c7.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.1.x-335-g9bdc2c7fac.

janisz mentioned this pull request Jun 30, 2023
Closed
5 tasks
janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_GetAll branch from 16b5b7d to a6dd70b Compare July 3, 2023 15:47
janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_Get branch from 0eaac20 to 8228546 Compare July 3, 2023 15:47
janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_GetAll branch from a6dd70b to 390bc82 Compare July 3, 2023 16:01
janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_Get branch from 8228546 to 3a85339 Compare July 3, 2023 16:01
janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_GetAll branch from 390bc82 to d6e5a0a Compare July 4, 2023 12:59
janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_Get branch from 3a85339 to b49fbe0 Compare July 4, 2023 12:59
janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_GetAll branch from d6e5a0a to a315299 Compare July 4, 2023 13:22
janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_Get branch from b49fbe0 to 54040b8 Compare July 4, 2023 13:22
janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_GetAll branch from a315299 to f501f43 Compare July 4, 2023 13:57
janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_Get branch from 54040b8 to 824a8e5 Compare July 4, 2023 13:57
rhybrillou reviewed Jul 5, 2023
View reviewed changes
Comment thread pkg/search/postgres/store.go Outdated
if ok, err := s.permissionChecker.GetAllowed(ctx); err != nil {
return nil, false, err
} else if !ok {
return nil, false, sac.ErrResourceAccessDenied

rhybrillou Jul 5, 2023

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

This changes the behaviour compared to the generated code.
I'd stick to the previous behaviour that returns nil, false, nil otherwise, this may leak information about the user allowed scope.

janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_GetAll branch from f501f43 to c14880c Compare July 6, 2023 11:02
janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_Get branch from 824a8e5 to 737eed5 Compare July 6, 2023 11:02
janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_GetAll branch from c14880c to 633cc54 Compare July 6, 2023 14:15
janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_Get branch from 737eed5 to f8ee78b Compare July 6, 2023 14:15
janisz requested a review from rhybrillou July 6, 2023 14:21
janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_GetAll branch from 633cc54 to 96d90a5 Compare July 6, 2023 15:36
janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_Get branch from f8ee78b to 467a72a Compare July 6, 2023 15:36

janisz commented Jul 7, 2023

Copy link
Copy Markdown
Contributor Author

@janisz started a stack merge that includes this pull request via Graphite.

janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_GetAll branch from 96d90a5 to ffe86f7 Compare July 7, 2023 08:08
Base automatically changed from master-janisz/06-30-ROX-18155_pg_generic_store_GetAll to master July 7, 2023 09:20

janisz commented Jul 7, 2023

Copy link
Copy Markdown
Contributor Author

Graphite rebased this pull request as part of a merge.

janisz force-pushed the master-janisz/06-30-ROX-18155_pg_generic_store_Get branch from 467a72a to 9bdc2c7 Compare July 7, 2023 09:20
janisz merged commit 6eb5fa2 into master Jul 7, 2023

janisz commented Jul 7, 2023

Copy link
Copy Markdown
Contributor Author

@janisz merged this pull request with Graphite.

janisz deleted the master-janisz/06-30-ROX-18155_pg_generic_store_Get branch July 7, 2023 10:17
This was referenced Jul 12, 2023
Merged
Merged
Merged
Merged
Closed
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

rhybrillou rhybrillou approved these changes

Assignees

No one assigned

Labels

area/central area/postgres

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL