Bug #37903
Uri respects query and fragment party only when path is present
| Status: | Resolved | Start date: | 2012-06-11 | |
|---|---|---|---|---|
| Priority: | Must have | Due date: | ||
| Assigned To: | Adrian Föder | % Done: | 100% |
|
| Category: | Http | |||
| Target version: | TYPO3 Flow Base Distribution - 1.1 beta 3 | |||
| PHP Version: | Complexity: | |||
| Has patch: | Yes | Affected Flow version: | Git 1.1 |
Description
With the current implementation, the query and fragment parts are only added to Uri's string representation if the path is set.
This means concretely, if
http://www.example.org?q=foo
is set, the query is omitted because the path is empty / not set.
As of http://tools.ietf.org/html/rfc3986#section-3 and http://tools.ietf.org/html/rfc3986#section-5.3, for example, the query and fragment may be present without the path part.
Associated revisions
[BUGFIX] Uri respects query and fragment even without path
Corresponding to the appropriate RFC 3986, section 3.3
the query and fragment indeed may be present also if no
path is given.
Change-Id: Ia118686cfb5673514fd39cddb85d0ecfaae98d99
Fixes: #37903
Releases: 1.1, 1.2
[BUGFIX] Uri respects query and fragment even without path
Corresponding to the appropriate RFC 3986, section 3.3
the query and fragment indeed may be present also if no
path is given.
Change-Id: Ia118686cfb5673514fd39cddb85d0ecfaae98d99
Fixes: #37903
Releases: 1.1, 1.2
History
#1 Updated by Gerrit Code Review about 3 years ago
- Status changed from New to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11929
#2 Updated by Gerrit Code Review about 3 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11929
#3 Updated by Gerrit Code Review about 3 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11929
#4 Updated by Karsten Dambekalns about 3 years ago
- Affected Flow version changed from Git 1.2 (master) to Git 1.1
#5 Updated by Gerrit Code Review about 3 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11929
#6 Updated by Gerrit Code Review about 3 years ago
Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11929
#7 Updated by Gerrit Code Review about 3 years ago
Patch set 1 for branch FLOW3-1.1 has been pushed to the review server.
It is available at http://review.typo3.org/12189
#8 Updated by Adrian Föder about 3 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 75d98528be52c83c6226b2eca162e166d4c98cc3.
#9 Updated by Karsten Dambekalns about 3 years ago
- Target version changed from 1.1 RC1 to 1.1 beta 3