Skip to content

Alter default privilege for snomedct schema - #1248

Merged
MartinWheelerMT merged 1 commit into
mainfrom
alter-default-priviliege-for-schema
Mar 9, 2026
Merged

Alter default privilege for snomedct schema#1248
MartinWheelerMT merged 1 commit into
mainfrom
alter-default-priviliege-for-schema

Conversation

@MartinWheelerMT

Copy link
Copy Markdown
Collaborator

What

What

Add table read permission to the default privileges for SnomedCt schema for the 'application_user' role

Why

  • In Postgres 17+ default privileges no longer grant usages on schemas.
  • The immunisation scripts are responsible for creating the schema and should grant the required access to the user role consuming them

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Internal change (non-breaking change with no effect on the functionality affecting end users)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • [] I have updated the Changelog with details of my change in the UNRELEASED section if this change will affect end users
  • A corresponding change has been made to the Mapping Documentation repository

@MartinWheelerMT
MartinWheelerMT requested a review from a team as a code owner March 9, 2026 14:13
@MartinWheelerMT
MartinWheelerMT enabled auto-merge (squash) March 9, 2026 14:18
@MartinWheelerMT
MartinWheelerMT force-pushed the alter-default-priviliege-for-schema branch from 7a9a5f7 to 883eb40 Compare March 9, 2026 14:58
@MartinWheelerMT
MartinWheelerMT force-pushed the alter-default-priviliege-for-schema branch from 883eb40 to 6c6aa48 Compare March 9, 2026 15:00
@github-actions

github-actions Bot commented Mar 9, 2026

Copy link
Copy Markdown

Looks good. No mutations were possible for these changes.
See https://pitest.org

@MartinWheelerMT
MartinWheelerMT merged commit eff1bd8 into main Mar 9, 2026
19 checks passed
@MartinWheelerMT
MartinWheelerMT deleted the alter-default-priviliege-for-schema branch March 9, 2026 15:21
@github-actions

github-actions Bot commented Mar 9, 2026

Copy link
Copy Markdown

Images built and published to ECR using a Build Id of PR-968-edd498d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants