Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Typo in documentation #660

Open
tstepien0 opened this issue Jul 4, 2024 · 3 comments
Open

Typo in documentation #660

tstepien0 opened this issue Jul 4, 2024 · 3 comments

Comments

@tstepien0
Copy link

tstepien0 commented Jul 4, 2024

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.

@tomgachet
Copy link

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!

@bersace
Copy link
Member

bersace commented Sep 9, 2024

Hi, thanks for the report.

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

I regenerated documentation at 2358dc0.

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,

@tomgachet
Copy link

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

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

No branches or pull requests

3 participants