wultra / powerauth-push-server

PowerAuth Push Server repository
Apache License 2.0
24 stars 12 forks source link

SQL error after DB column escaping #861

Closed romanstrobl closed 3 weeks ago

romanstrobl commented 1 month ago

The column escaping introduced in https://github.com/wultra/powerauth-push-server/pull/859/files does not work properly on Oracle due to enforcing case sensitivity.

We should probably go back to the original solution which was renaming columns, which has no side effects, it only slightly complicates the migration to new release.

Log:

app-1      | 2024-07-03T13:08:54.639Z ERROR 218 --- [powerauth-java-server] [io-8080-exec-14] [0395b0626facc7544e7e4fc1383731fe-199b8d50fd8766c9] o.h.engine.jdbc.spi.SqlExceptionHelper   : ORA-00904: "type": invalid identifier
app-1      |
app-1      | https://docs.oracle.com/error-help/db/ora-00904/
app-1      | 2024-07-03T13:08:54.646Z ERROR 218 --- [powerauth-java-server] [io-8080-exec-14] [0395b0626facc7544e7e4fc1383731fe-199b8d50fd8766c9] i.g.p.e.DefaultExceptionHandler          : could not execute statement [ORA-00904: "type": invalid identifier
app-1      |
app-1      | https://docs.oracle.com/error-help/db/ora-00904/] [insert into push_inbox (body,inbox_id,"read",subject,summary,timestamp_created,timestamp_read,"type",user_id,id) values (?,?,?,?,?,?,?,?,?,?)]; SQL [insert into push_inbox (body,inbox_id,"read",subject,summary,timestamp_created,timestamp_read,"type",user_id,id) values (?,?,?,?,?,?,?,?,?,?)]
app-1      |
app-1      | org.springframework.dao.InvalidDataAccessResourceUsageException: could not execute statement [ORA-00904: "type": invalid identifier
app-1      |
OliverJanstaWultra commented 2 weeks ago

@korbelm Please test this one