ldap2pg icon indicating copy to clipboard operation
ldap2pg copied to clipboard

Typo in documentation

Open tstepien0 opened this issue 1 year ago • 3 comments

In the documentation regarding built-in privileges in section "Profiles reference", both "usage_on_schemas" and "usage_on_schema" appear. Only the first parameter is correct; the second one can be misleading and doesn't work.

tstepien0 avatar Jul 04 '24 13:07 tstepien0

You're right @tstepien0 ! The error is actually in the Profile all_on_schemas section. The privilege __usage_on_schema__ must be __usage_on_schemas__ to be functional!

tomgachet avatar Sep 06 '24 16:09 tomgachet

Hi, thanks for the report.

Actually, this was a bug in configuration defaults fixed in 643f1387a5d60b469df1184ebebd7fdb8fc18ed0 . Which mean 6.1 has a bug in default configuration. __all_on_schemas__ does not effectively include __usage_on_schemas__.

I regenerated documentation at 2358dc01161ea11ae196c338def1af7a86998c4c.

Please keep the issue open so I track a final task on this issue : prevent regression by controlling consistency of default configuration, add a CI job to ensure generated docs is up2date with the code.

Thanks @tomgachet for the head-up.

Regards,

bersace avatar Sep 09 '24 06:09 bersace

Thanks @bersace , it's a pleasure to contribute modestly to ldap2pg!

tomgachet avatar Sep 09 '24 07:09 tomgachet

Fixed in 6.2

bersace avatar Dec 12 '24 10:12 bersace