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

Workaround to query ENA / ERAPRO Views. #333

Merged
merged 1 commit into from
Nov 1, 2023

Conversation

mberacochea
Copy link
Member

ERA needs to be appended as the default connection tries to use the PUBLIC SYNONYM (according to ENA) and it's not working ATM we were advised to prefix the views and this is the simplest way. The short-term plan is to remove the dependency of ENA databases

ERA needs to be appended as the default connection tries to use
the PUBLIC SYNONYM (according to ENA) and it's not working ATM
we were advised to prefix the views and this is the simplest way.
The short-term plan is to remove the dependency of ENA databases
Copy link
Member

@SandyRogers SandyRogers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @mberacochea .
Is it possible to use the db_tablespace attribute instead of this, to rely slightly less on the quote mark injection?

@mberacochea
Copy link
Member Author

Thank you, @mberacochea . Is it possible to use the db_tablespace attribute instead of this, to rely slightly less on the quote mark injection?

it doesn't work, I think an elegant solution would be to use a db router to change the schema on the "fly"

Copy link
Contributor

@KateSakharova KateSakharova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@mberacochea mberacochea merged commit a33851f into master Nov 1, 2023
4 checks passed
@SandyRogers
Copy link
Member

Thank you, @mberacochea . Is it possible to use the db_tablespace attribute instead of this, to rely slightly less on the quote mark injection?

it doesn't work, I think an elegant solution would be to use a db router to change the schema on the "fly"

Interesting; thanks for checking.

@mberacochea mberacochea deleted the hotfix/webuploader-ena-db-views-query-fix branch November 1, 2023 10:42
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 this pull request may close these issues.

3 participants