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

django.db.utils.IntegrityError on Oracle for silk.models.Request. #323

Open
danizen opened this issue Dec 20, 2018 · 3 comments · May be fixed by #619
Open

django.db.utils.IntegrityError on Oracle for silk.models.Request. #323

danizen opened this issue Dec 20, 2018 · 3 comments · May be fixed by #619

Comments

@danizen
Copy link

danizen commented Dec 20, 2018

django 1.11.16
django-silk 3.0.1

Oracle makes no difference between '' and BLANK, and so the file field in silk.models.Request does not work properly. The field is declared as:
prof_file = FileField(max_length=300, blank=True, storage=silk_storage)

On the oracle level, I have the following constraint for this field:
PROF_FILE NVARCHAR2(300) default NULL not null

Since you are just using django's FileField you may call it a Django error. But you should track it, anyway.

My solution may be on local to use my database router to route silk to an sqlite3 database. This solution, provided it worked, could be documented somewhere.

@danizen
Copy link
Author

danizen commented Dec 20, 2018

Using a local sqlite3 database via a database router addresses this issue, but it is worth fixing.

@nasirhjafri
Copy link
Member

Hi @danizen Can you please attach a minimal app producing the error?

@SebCorbin
Copy link
Contributor

Should be solved by #258

@SebCorbin SebCorbin linked a pull request Oct 31, 2022 that will close this issue
@SebCorbin SebCorbin linked a pull request Oct 31, 2022 that will close this issue
@SebCorbin SebCorbin removed the has PR label Nov 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants