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

Pair of requests to the wishlist #60

Open
jcpsaravia opened this issue Nov 25, 2019 · 2 comments
Open

Pair of requests to the wishlist #60

jcpsaravia opened this issue Nov 25, 2019 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@jcpsaravia
Copy link

  1. I know that Postgres is case sensitive, but is it possible for the tool to specify PostgreSQL to maintain the lower-upper case present in the gonymizer configuration file when creating the database in which the data will be loaded.

  2. Could you add a flag/mechanism in the gonymizer config file so that the pg_dump command does not specify the schema flag? I had cases in which extensions are not being dumped in the backup file and this is because how the schema flag works

@junkert junkert self-assigned this Nov 26, 2019
@junkert junkert added the enhancement New feature or request label Nov 26, 2019
@junkert
Copy link
Collaborator

junkert commented Nov 26, 2019

I'll try to get to this task this week.

  1. Just need to verify I understand this one correctly. Are you saying that the database name that is given to Gonymizer in the Gonymizer config is always lowercased for the "load" operation? Guessing this:

Incorrect:

"database": "MyCamelCase"  => mycamelcase

Correct:

"database": "MyCamelCase" => MyCamelCase
  1. The function issue above is an interesting one considering pg_dump documentation is sorely lacking on how it includes functions at the schema level. This will take some research and digging into the documentation to figure it out, but this is an area that needs improved in Gonymizer. More than likely we are implementing this incorrectly.

Thanks for submitting this bug report @jcpsaravia

@jcpsaravia
Copy link
Author

jcpsaravia commented Dec 1, 2019

With regards to your doubts:

  1. Yes you are correct. The issue lies in the fact that if the database (where the anonymized data is to be loaded) is written with capital letters in the configuration file, it is created with lower case letters by PostgreSQL.

  2. I just found a work around for this case(not an elegant solution). I basically commented line 71 of the generator.go file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants