WIP: move get_current_url() and co in OmniTools Request class
Changes PlannedPublic
Actions

Authored by dereckson on Nov 11 2025, 18:13.

Tags

None

Referenced Files

	F24941315: D3881.id10049.diff
	Fri, Mar 20, 09:30

	Unknown Object (File)
	Sun, Mar 15, 01:25

	Unknown Object (File)
	Fri, Mar 13, 19:05

	Unknown Object (File)
	Thu, Mar 12, 16:24

	Unknown Object (File)
	Wed, Mar 11, 23:14

	Unknown Object (File)
	Wed, Mar 11, 10:55

	Unknown Object (File)
	Wed, Mar 11, 00:37

	Unknown Object (File)
	Wed, Mar 11, 00:15

View All Files

Subscribers

None

Details

Reviewers: None

Test Plan

This code has been battle-tested with several sites using:

direct .php access in the URL
Apache + mod_rewrite
nginx try_files directive

Diff Detail

Repository

rKERUALD Keruald libraries development repository

Lint

Lint Passed

Unit

Test Failures

Branch

request-current-url

Build Status

Buildable 6184
Build 6468: arc lint + arc unit

Unit TestsBroken
View All

	Time	Test
	0 ms	/datacube/dereckson/dev/nasqueron/keruald/keruald/omnitools/tests/HTTP/Requests/RequestTest.php

Event Timeline

dereckson requested review of this revision.Nov 11 2025, 18:13

dereckson created this revision.

Harbormaster completed remote builds in B6184: Diff 10049.Nov 11 2025, 18:13

Unit tests still to commit

Revision Contents
Changeset List

Path

Size

omnitools/

src/

HTTP/

Requests/

Request.php

1 line

WithLocalURL.php

141 lines

Diff 10049

View Options

omnitools/src/HTTP/Requests/Request.php