Skip to content

RequestFactory: rejects invalid URL [WIP]#41

Open
dg wants to merge 2 commits intonette:masterfrom
dg:pull-invalid
Open

RequestFactory: rejects invalid URL [WIP]#41
dg wants to merge 2 commits intonette:masterfrom
dg:pull-invalid

Conversation

@dg
Copy link
Member

@dg dg commented Dec 27, 2014

See #30

@JanTvrdik
Copy link
Contributor

👍 This is exactly what I had in mind. Now we just need to figure out how to handle the invalid request 😄


Partially related to it – what should RequestFactory do when running in CLI? Because that is another form of invalid URL (or HTTP request in general).


You can remove the Strings::fixEncoding now as well.

@dg dg force-pushed the master branch 2 times, most recently from 6df6a37 to f17b437 Compare February 9, 2015 23:14
@dg dg added this to the v2.4 milestone Feb 17, 2015
@dg dg force-pushed the pull-invalid branch from 5fc99ff to b45ad0c Compare May 25, 2015 13:27
@dg dg force-pushed the pull-invalid branch 3 times, most recently from a2644ca to e76600e Compare June 19, 2015 17:03
@dg dg force-pushed the master branch 7 times, most recently from 1dc599c to f35d20f Compare November 5, 2015 00:41
@dg dg force-pushed the master branch 2 times, most recently from 7981e33 to 3fb64f5 Compare July 21, 2016 12:07
@dg dg force-pushed the master branch 3 times, most recently from b9f06ea to 4f6d6c9 Compare December 20, 2016 14:30
@dg dg force-pushed the master branch 8 times, most recently from a00de7a to 4210d9e Compare January 30, 2017 01:57
@dg dg force-pushed the master branch 2 times, most recently from 7f12fde to 9791354 Compare February 10, 2017 12:57
@dg dg force-pushed the master branch 2 times, most recently from 51f373c to 69af026 Compare June 13, 2017 09:59
@dg dg force-pushed the master branch 3 times, most recently from 87fefb5 to 9590f91 Compare July 21, 2017 19:42
@dg dg force-pushed the master branch 2 times, most recently from 898e948 to b550f69 Compare March 8, 2018 12:29
@dg dg force-pushed the master branch 2 times, most recently from 56c2641 to 544d9e6 Compare March 14, 2018 18:37
@dg dg force-pushed the master branch 5 times, most recently from 0ac4cf7 to c1cfba7 Compare August 28, 2018 14:10
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