Skip to content

escape "_" when using LIKE operator #311

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

Closed
wants to merge 1 commit into from
Closed

Conversation

ioguix
Copy link
Contributor

@ioguix ioguix commented Jun 8, 2023

Hi,

Function schemas_are is currently with extension pgsodium because of bad escaping of _ with the LIKE operator.

Here is a small fix.

Thanks,

ioguix added a commit to ioguix/pgsodium that referenced this pull request Jun 8, 2023
@theory theory self-assigned this Jun 8, 2023
@theory theory added the bug label Jun 8, 2023
michelp pushed a commit to michelp/pgsodium that referenced this pull request Jun 8, 2023
* Fix test error on schema caused by pgtap bug

Bug reported to pgtap upstream here:

  theory/pgtap#311

* Test minimal postgresql supported version

* Fix test failing because of objects order

the test can fail when running the test on a database with
a different collation than the one used to write the test.

* Fix SQL error with pg 16

* Fix test errors with pg_read_all_data and pg_write_all_data roles

* Update pgsodium version in test generator

* Add tests about masking_rule.security_invoker column

* Comply tests generator and SQL schema to the new test.sql script

* Create role and extension inside the test's transaction

* Create pgsodium extension outside of the transaction

When creating the extension inside the xact, one test is failing with
the following error:

  ERROR:  unsafe use of new value "secretbox" of enum type key_type
  LINE 1: SELECT id as secretbox_key_id from create_key('secretbox')
                                                      ^
  HINT:  New enum values must be committed before they can be used

* Make volumes accessible inside the container with podman
@theory
Copy link
Owner

theory commented Jun 10, 2023

Thank you! Superseded by #312.

@theory theory closed this Jun 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants