Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[5.1] Use write database connection when validating uniqueness#13718

Merged
taylorotwell merged 1 commit into
laravel:5.1from
crynobone:patch/validation-use-write-pdo
May 26, 2016
Merged

[5.1] Use write database connection when validating uniqueness#13718
taylorotwell merged 1 commit into
laravel:5.1from
crynobone:patch/validation-use-write-pdo

Conversation

@crynobone
Copy link
Copy Markdown
Member

It would be possible that when an application have a huge latency
between master and slave that a novice user would reattempt to recreate
an entry which is now only available on the master connection.

When he/she create the record again which require unique value such as email
it currently would only fetch from the slave database connection and
would return pass, however when the application try to save the actual
entry to the database it would fail due to unique (if we set it on the
database).

This solve the issue by always checking against master (write)
connection.

Signed-off-by: crynobone [email protected]

It would be possible that when an application have a huge latency
between master and slave that a novice user would reattempt to recreate
an entry which is now only available on the master connection.

When he/she create the record again which require unique value such as email
it currently would only fetch from the slave database connection and
would return pass, however when the application try to save the actual
entry to the database it would fail due to unique (if we set it on the
database).

This solve the issue by always checking against master (write)
connection.

Signed-off-by: crynobone <[email protected]>
@crynobone
Copy link
Copy Markdown
Member Author

This is almost similar with #12794 bugfix.

@GrahamCampbell GrahamCampbell changed the title Use write database connection when validating uniqueness [5.1] Use write database connection when validating uniqueness May 26, 2016
@taylorotwell taylorotwell merged commit c688e5a into laravel:5.1 May 26, 2016
@crynobone crynobone deleted the patch/validation-use-write-pdo branch April 19, 2017 00:21
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.

2 participants