Opened 2 years ago
Last modified 2 months ago
#16483 new defect (bug)
Visibility: password-protected exposes multiple pages
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Future Release |
| Component: | Security | Version: | 3.0.4 |
| Severity: | minor | Keywords: | has-patch dev-feedback needs-testing |
| Cc: | mdhansen@… |
Description
- password protect a page ('protected') with a password
- password protect another page ('thistoo') with the SAME password
- visit 'protected' and enter the password. Page is visible
- visit 'thistoo'; expected: prompt for password. What happens: Page is visible
Regardless of whether someone with a password has the right to try it in as many pages as they want (and would therefore successfully see the page if the passwords were the same), the user should still be prompted on a page-by-page basis. Global authentication to multiple pages is possible with user accounts and roles. It should not be possible with visibility: password-protected pages.
Attachments (5)
Change History (12)
solarissmoke — 2 years ago
comment:1
solarissmoke — 2 years ago
- Keywords has-patch dev-feedback added
I agree that this shouldn't happen - although plain text passwords in cookies aren't really going to prevent someone who is determined ;)
Here's one possible patch. It may cause some issues because previously (and since WP 1.0.0) get_the_password_form() didn't require a post. I can't find any instances in core that don't have a post (or implicit post global) set, but I might have missed something.
comment:2
MikeHansenMe — 6 months ago
- Cc mdhansen@… added
- Keywords needs-refresh added
This problem still exists in 3.5-RC1-22924. Patch needs refresh.
SergeyBiryukov — 6 months ago
comment:3
SergeyBiryukov — 6 months ago
- Keywords needs-testing added; needs-refresh removed
comment:4
follow-up:
↓ 5
MikeHansenMe — 6 months ago
16483.3.diff does not seem to let me log in to the page at all even with the correct password.
SergeyBiryukov — 4 months ago
comment:5
in reply to:
↑ 4
SergeyBiryukov — 4 months ago
Replying to MikeHansenMe:
16483.3.diff does not seem to let me log in to the page at all even with the correct password.
Thanks, fixed in 16483.4.diff. Missed [19728] in the previous patch.
comment:6
SergeyBiryukov — 4 months ago
- Component changed from General to Security
- Milestone changed from Awaiting Review to 3.6
Related: #19797

Make passwords post-specific