Skip to content

fix: #769 drop fileds added by pg13, superuser is enough#810

Closed
yihong0618 wants to merge 1 commit intopostgresml:masterfrom
yihong0618:master
Closed

fix: #769 drop fileds added by pg13, superuser is enough#810
yihong0618 wants to merge 1 commit intopostgresml:masterfrom
yihong0618:master

Conversation

@yihong0618
Copy link

No description provided.

Signed-off-by: yihong0618 <zouzou0208@gmail.com>
@levkk
Copy link
Contributor

levkk commented Jul 12, 2023

trusted = true is needed for the extension to be CREATE-able by a non-superuser. I need to fix this in pg11 & pg12 though, where trusted is not supported, as you mentioned. I think we'll just conditionally remove it for those versions. The extension still requires superuser because it's a C extension I believe. Please correct me if I'm wrong.

@liuxueyang
Copy link

liuxueyang commented Nov 17, 2023

I guess it makes more sense to do this in pgrx. Already created PR for pgrx

@liuxueyang
Copy link

The PR for pgrx pgcentralfoundation/pgrx#1397 was merged and I guess this PR can be closed. 😄

@montanalow montanalow closed this Jan 29, 2024
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.

4 participants