Skip to content

BI-2833 - Fix security gap, only system admins to pull data across programs - #548

Merged
humsika merged 6 commits into
release/1.5from
feature/BI-2833
Sep 1, 2026
Merged

humsika merged 6 commits into
release/1.5from
feature/BI-2833

Conversation

@nickpalladino

@nickpalladino nickpalladino commented Aug 25, 2026

Copy link
Copy Markdown
Member

Description

Story: BI-2833

  • updated /serverinfo, removed passthrough calls only available to system admins
  • changed permissions on passthrough calls to only be accessible to system admins

Dependencies

  • develop java-brapi-ProdServer

Testing

See acceptance criteria in card

Checklist:

  • I have performed a self-review of my own code
  • I have tested my code and ensured it meets the acceptance criteria of the story
  • I have create/modified unit and/or integration tests to cover this change or tests are not applicable
  • I have commented my code, particularly in hard-to-understand areas
  • I have either updated the source of truth or arranged for update with product owner if needed https://breedinginsight.atlassian.net/wiki/spaces/BI/pages/1559953409/Source+of+Truth

@nickpalladino nickpalladino changed the title Updated /serverinfo BI-2833 - Fix security gap, only system admins to pull data across programs Aug 27, 2026
@nickpalladino
nickpalladino marked this pull request as ready for review August 31, 2026 19:26
@nickpalladino
nickpalladino requested review from a team, HMS17 and humsika and removed request for a team and HMS17 August 31, 2026 19:59

@humsika humsika 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.

Code looks good to me and is working as expected.

@humsika
humsika merged commit bcb89ab into release/1.5 Sep 1, 2026
1 check passed
@humsika
humsika deleted the feature/BI-2833 branch September 1, 2026 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants