Commit0fef877

committed

Rename session_auth_is_superuser to current_role_is_superuser.

This variable might've been accurately named when it was added inea88633, but the name hasn't been accurate since at least theintroduction of SET ROLE ine5d6b91. The correspondingdocumentation was fixed ineedb068. This commit renames thevariable accordingly.Suggested-by: Joseph KoshakowDiscussion:https://postgr.es/m/CAAvxfHc-HHzONQ2oXdvhFF9ayRnidPwK%2BfVBhRzaBWYYLVQL-g%40mail.gmail.com

1 parent411b720 commit0fef877Copy full SHA for 0fef877

File tree

-5

lines changed

-5

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -327,7 +327,7 @@ InitializeParallelDSM(ParallelContext *pcxt)`
`327`	`327`	`fps->database_id=MyDatabaseId;`
`328`	`328`	`fps->authenticated_user_id=GetAuthenticatedUserId();`
`329`	`329`	`fps->outer_user_id=GetCurrentRoleId();`
`330`		`-fps->is_superuser=session_auth_is_superuser;`
	`330`	`+fps->is_superuser=current_role_is_superuser;`
`331`	`331`	`GetUserIdAndSecContext(&fps->current_user_id,&fps->sec_context);`
`332`	`332`	`GetTempNamespaceState(&fps->temp_namespace_id,`
`333`	`333`	`&fps->temp_toast_namespace_id);`

Lines changed: 6 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -511,7 +511,7 @@ boolcheck_function_bodies = true;`
`511`	`511`	`* details.`
`512`	`512`	`*/`
`513`	`513`	`booldefault_with_oids= false;`
`514`		`-boolsession_auth_is_superuser;`
	`514`	`+boolcurrent_role_is_superuser;`
`515`	`515`
`516`	`516`	`intlog_min_error_statement=ERROR;`
`517`	`517`	`intlog_min_messages=WARNING;`
`@@ -1037,13 +1037,16 @@ struct config_bool ConfigureNamesBool[] =`
`1037`	`1037`	`NULL,NULL,NULL`
`1038`	`1038`	`},`
`1039`	`1039`	`{`
`1040`		`-/* Not for general use --- used by SET SESSION AUTHORIZATION */`
	`1040`	`+/*`
	`1041`	`+ * Not for general use --- used by SET SESSION AUTHORIZATION and SET`
	`1042`	`+ * ROLE`
	`1043`	`+ */`
`1041`	`1044`	`{"is_superuser",PGC_INTERNAL,UNGROUPED,`
`1042`	`1045`	`gettext_noop("Shows whether the current user is a superuser."),`
`1043`	`1046`	`NULL,`
`1044`	`1047`	`GUC_REPORT \|GUC_NO_SHOW_ALL \|GUC_NO_RESET_ALL \|GUC_NOT_IN_SAMPLE \|GUC_DISALLOW_IN_FILE`
`1045`	`1048`	`},`
`1046`		`-&session_auth_is_superuser,`
	`1049`	`+&current_role_is_superuser,`
`1047`	`1050`	`false,`
`1048`	`1051`	`NULL,NULL,NULL`
`1049`	`1052`	`},`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -250,7 +250,7 @@ extern PGDLLIMPORT bool log_statement_stats;`
`250`	`250`	`externPGDLLIMPORTboollog_btree_build_stats;`
`251`	`251`
`252`	`252`	`externPGDLLIMPORTboolcheck_function_bodies;`
`253`		`-externPGDLLIMPORTboolsession_auth_is_superuser;`
	`253`	`+externPGDLLIMPORTboolcurrent_role_is_superuser;`
`254`	`254`
`255`	`255`	`externPGDLLIMPORTboollog_duration;`
`256`	`256`	`externPGDLLIMPORTintlog_parameter_max_length;`

Comments

(0)