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

Escape column_name in hypertransformer #110

Closed
csala opened this issue Jun 25, 2020 · 0 comments · Fixed by #111
Closed

Escape column_name in hypertransformer #110

csala opened this issue Jun 25, 2020 · 0 comments · Fixed by #111
Assignees
Milestone

Comments

@csala
Copy link
Contributor

csala commented Jun 25, 2020

The column name is being used in the HyperTransformer class to generate a regular expression that later on is used to identify which transformed columns need to be used to revert the transformation:

regex = r'{}(#[0-9]+)?$'.format(column_name)

This column name should be escaped to avoid conflicts when there are symbols that can be interpreted as regex escape sequences.

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

Successfully merging a pull request may close this issue.

1 participant