[openstreetmap/openstreetmap-website] Add openid connect support using doorkeeper-openid_connect gem (PR #4226)

[Milan Cvetkovic]

@milan-cvetkovic commented on this pull request.



> @@ -0,0 +1,5 @@
+class ValidateCreateDoorkeeperOpenidConnectTables < ActiveRecord::Migration[7.0]

Hm, db:migrate was failing due to possible deadlock when creating foreign keys and validating them in one shot, and suggested splitting to two migration scripts, which I did. I guess I could merge them into one.

Here is the error message from db:migrate

> === Dangerous operation detected #strong_migrations ===
>
> Adding a foreign key blocks writes on both tables. Instead,
> add the foreign key without validating existing rows,
> then validate them in a separate migration.


-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4226#discussion_r1315049383
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/4226/review/1609710959 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20230904/ce850e47/attachment.htm>