Skip to content

[#823] Allow RULE_EXEC_CONTEXT column access by GenQuery1 - #829

Merged
korydraughn merged 1 commit into
irods:mainfrom
d-w-moore:823.m
Sep 4, 2026
Merged

[#823] Allow RULE_EXEC_CONTEXT column access by GenQuery1#829
korydraughn merged 1 commit into
irods:mainfrom
d-w-moore:823.m

Conversation

@d-w-moore

Copy link
Copy Markdown
Collaborator

No description provided.

@trel trel changed the title Allow RULE_EXEC_CONTEXTZ column access by GenQuery1 Allow RULE_EXEC_CONTEXT column access by GenQuery1 Aug 28, 2026
Comment thread irods/test/rule_test.py Outdated
Comment thread irods/test/rule_test.py Outdated
Comment thread irods/test/rule_test.py Outdated
Comment thread irods/test/rule_test.py Outdated
Comment thread irods/test/rule_test.py Outdated
Comment thread irods/test/rule_test.py Outdated

@korydraughn korydraughn left a comment

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.

Looks good to me.

Is this ready for squashing?

@d-w-moore

Copy link
Copy Markdown
Collaborator Author

Ok, good the Github machinery finally rose in the right constellation and blessed this PR. The only exception is codacy's flagging of the cryptographic issue, which is ok as luckily the randint function isn't being used for cryptography.

@d-w-moore

Copy link
Copy Markdown
Collaborator Author

Looks good to me.

Is this ready for squashing?

think so.

@d-w-moore
d-w-moore force-pushed the 823.m branch 2 times, most recently from ef30c49 to 8205710 Compare September 2, 2026 18:05
@d-w-moore

Copy link
Copy Markdown
Collaborator Author

Force-pushed after the squash so the commit message contains the explicit DB column being queried.

@korydraughn korydraughn left a comment

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.

Pound it.

@d-w-moore d-w-moore changed the title Allow RULE_EXEC_CONTEXT column access by GenQuery1 [#823] Allow RULE_EXEC_CONTEXT column access by GenQuery1 Sep 4, 2026
@d-w-moore

Copy link
Copy Markdown
Collaborator Author

Ok, #'d

@korydraughn
korydraughn merged commit 01c4794 into irods:main Sep 4, 2026
11 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants