Commite8d9caa

committed

Accept invalidation messages in InitializeSessionUserId().

If the authentication method modified the system catalogs through aseparate database connection (say, to create a new role on the fly),make sure syscache sees the changes before we try to find the user.Author: Thomas MunroReviewed-by: Tom Lane, Andres FreundDiscussion:https://postgr.es/m/CAEepm%3D3_h0_cgmz5PMyab4xk_OFrg6G5VCN%3DnF4chFXM9iFOqA%40mail.gmail.com

1 parent387a5cf commite8d9caaCopy full SHA for e8d9caa

File tree

-0

lines changed

-0

lines changed

Lines changed: 8 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -48,6 +48,7 @@`
`48`	`48`	`#include"storage/procarray.h"`
`49`	`49`	`#include"utils/builtins.h"`
`50`	`50`	`#include"utils/guc.h"`
	`51`	`+#include"utils/inval.h"`
`51`	`52`	`#include"utils/memutils.h"`
`52`	`53`	`#include"utils/pidfile.h"`
`53`	`54`	`#include"utils/syscache.h"`
`@@ -594,6 +595,13 @@ InitializeSessionUserId(const char *rolename, Oid roleid)`
`594`	`595`	`/* call only once */`
`595`	`596`	`AssertState(!OidIsValid(AuthenticatedUserId));`
`596`	`597`
	`598`	`+/*`
	`599`	`+ * Make sure syscache entries are flushed for recent catalog changes.`
	`600`	`+ * This allows us to find roles that were created on-the-fly during`
	`601`	`+ * authentication.`
	`602`	`+ */`
	`603`	`+AcceptInvalidationMessages();`
	`604`	`+`
`597`	`605`	`if (rolename!=NULL)`
`598`	`606`	`{`
`599`	`607`	`roleTup=SearchSysCache1(AUTHNAME,PointerGetDatum(rolename));`

Comments

(0)