Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

#15209 closed (duplicate)

Manager.raw() allow for WITH ( ... ) SELECT

Reported by: Mike Fogel Owned by: nobody
Component: Database layer (models, ORM) Version: dev
Severity: Keywords: with, Manager.raw()
Cc: Mike Fogel Triage Stage: Accepted
Has patch: yes Needs documentation: yes
Needs tests: yes Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

The RawQuery object used by Manager.raw() validates input SQL to ensure that only SELECT queries can be ran through it. This is done such that SELECT queries with using a WITH (...) clause fail validation.

I propose changing the validation of the SQL such that SELECTs with these WITH clauses are ok.

The attached patch does that. It also does not weaken the validation as preceding WITH clauses can only be used with SELECT statements. Re:

Attachments (1)

with_clause_allow.diff (620 bytes ) - added by Mike Fogel 14 years ago.

Download all attachments as: .zip

Change History (4)

by Mike Fogel, 14 years ago

Attachment: with_clause_allow.diff added

comment:1 by Russell Keith-Magee, 14 years ago

Needs documentation: set
Needs tests: set
Patch needs improvement: set
Triage Stage: UnreviewedAccepted

Accepted on principle; a WITH clause is easy to identify,

However, if all backends don't support a feature, then validation should be a backend-specific feature.

In addition, the patch requires tests and documentation updates.

comment:2 by Ramiro Morales, 14 years ago

Resolution: duplicate
Status: newclosed

Duplicate of #14733.

comment:3 by Jacob, 13 years ago

milestone: 1.4

Milestone 1.4 deleted

Note: See TracTickets for help on using tickets.
Back to Top