Skip to content

Conversation

adamretter
Copy link
Contributor

@adamretter adamretter commented Jun 20, 2022

This adds better security controls for accessing Environment Variables and Java System Properties.

  • Fixed the return cardinality type of fn:environment-variable#1.
  • fn:available-environment-variables#0 and fn:enviroment-variable#1 are now deterministic.
  • Added the function util:available-system-properties#0.
  • util:available-system-properties#0 and util:system-property#1 are now deterministic.
  • Allow access to specific Environment Variables when using fn:available-environment-variables#0 and fn:environment-variable#1 to be configured from conf.xml
  • Allow access to specific Java System Properties when using fn:available-system-properties#0 and fn:system-property#1 to be configured from conf.xml

@adamretter adamretter added bug issue confirmed as bug enhancement new features, suggestions, etc. labels Jun 20, 2022
@adamretter adamretter added this to the eXist-4.10.1 milestone Jun 20, 2022
@adamretter adamretter requested a review from a team June 20, 2022 22:35
@adamretter adamretter marked this pull request as ready for review June 21, 2022 15:00
Copy link
Member

@ljo ljo left a comment

Choose a reason for hiding this comment

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

Looks good.

@line-o line-o added the needs documentation Signals issues or PRs that will require an update to the documentation repo label Jun 21, 2022
@line-o line-o removed the needs documentation Signals issues or PRs that will require an update to the documentation repo label Jun 22, 2022
@adamretter adamretter marked this pull request as draft August 15, 2022 17:46
@adamretter adamretter modified the milestones: eXist-4.10.1, eXist-4.11.1 Jan 11, 2023
@adamretter adamretter modified the milestones: eXist-4.11.1, eXist-4.11.3 Oct 17, 2024
@line-o
Copy link
Member

line-o commented Aug 3, 2025

I propose to close this PR without further action as exist-db v4 is EOL and no longer maintained

@line-o line-o added the close-without-action Proposed to be closed without further action. This will be voted on on the next community call. label Aug 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug issue confirmed as bug close-without-action Proposed to be closed without further action. This will be voted on on the next community call. enhancement new features, suggestions, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants