161 Commits (master)

Author	SHA1	Message	Date
n0nag0n	1e4c07da52	correct unit test and removed final keywords	1 year ago
n0nag0n	593e1ee9bc	phpstan cleanup	1 year ago
n0nag0n	83d33348e0	old array syntax cleanup and legacy json serializable removal	1 year ago
n0nag0n	2d4c7d4bcd	a few cleanup comments	1 year ago
n0nag0n	bef92303d7	Merge pull request #514 from flightphp/middleware ... Middleware code	1 year ago
n0nag0n	b388a26765	fixed alias issue, levenshtein recommendations and coverage-check	1 year ago
Austin Collier	f543d74bfe	Middleware code	1 year ago
Austin Collier	c7a143db04	fixed issue with multiline possibilities in url query	1 year ago
Austin Collier	8bd7fd035b	fixed params with numbers	1 year ago
Austin Collier	74d2fd7002	Fixes for group routing	1 year ago
Austin Collier	387f15bd8b	fixed optional params with aliasing	1 year ago
n0nag0n	7991530761	Route alias and phpstan updates	1 year ago
Austin Collier	65b3d5445f	Lots more unit testing for group based routing	1 year ago
Austin Collier	12073629cc	added ability to group routes together	1 year ago
n0nag0n	7b15d2cfca	100% coverage for net classes	1 year ago
n0nag0n	a416bfe19b	100% coverage for Request and Route	1 year ago
n0nag0n	0d026b16d1	added test for unique subdir setup in request	1 year ago
fadrian06	42ec161d8a	Added code format	1 year ago
fadrian06	fbcc9108c2	DocBlocks improved	1 year ago
Mike Cao	a130231646	Merge pull request #456 from jamalo/master ... PHP 8.1: Type declare length as int	3 years ago
Mike Cao	fc43bfb4ea	Update flight/net/Request.php ... Co-authored-by: Michal Hybner <76526074+mu1f407@users.noreply.github.com>	3 years ago
Michal Hybner	21a0c55375	Fix JSON request with empty body	3 years ago
Jamal	d2a92aa808	Fix PHP 8.1: Assign string to int propety ... TypeError: Cannot assign string to property flight\net\Request::$length of type int	3 years ago
Masroor Ehsan	dd73a61c32	FIX: Typed property flight\net\Response::$body must not be accessed before initialization	3 years ago
Masroor Ehsan	b7abb4945f	check for valid json input	3 years ago
Masroor Ehsan	a2c5dd5ede	minor refactorings	4 years ago
Masroor Ehsan	5b8e743e3d	refactorings	4 years ago
Masroor Ehsan	d3feb77ce9	fixed parameter types	4 years ago
Masroor Ehsan	04e471bf46	PHP ^7.4|^8.0 compatibility	4 years ago
Mike Cao	b6429d2a3c	Merge pull request #433 from juneszh/master ... add content_length configuration	4 years ago
juneszh	ed9bbed287	add content_length configuration	4 years ago
Mike Cao	b3120f8db2	Merge pull request #409 from pluveto/patch-1 ... Trim url	5 years ago
pluveto	ddb806a604	Trim url pattern ... This helps support whitespaces between METHOD and URL: ```php Flight::route('GET /auth/nonce', array($apiAuth, 'getNonce')); // `GET[space][space]/auth/nonce` Flight::route('POST /auth/login/email', array($apiAuth, 'loginByEmail')); ```	5 years ago
Andreas Kugel	0145f629d3	Enable DELETE method to receive request body ... Delete for CRUD needs request data in body	5 years ago
Mike Cao	da8eaefc63	Merge pull request #398 from hlidotbe/feature/request-host ... Add host to request	6 years ago
Hugues Lismonde	b4c9c42b00	Add host to request ... The requested host can be useful to build an url or route request based on which host was used.	6 years ago
Hugues Lismonde	e0dfb7406d	Fix request schem ... As mentionned in #396, the request scheme was not as documented, returning SERVER_PROTOCOL instead. The `getScheme` function is now used to handle common cases (HTTPS, FORWATED_PROTO, ...). `request->secure` is also based on the scheme now as `$_SERVER['HTTPS']` is unreliable for this purpose.	6 years ago
Delyan Angelov	0daa3f81b5	Allow cyrillic letters in the routes, by matching the result of urldecode over the URL .	6 years ago
Mike Cao	e146b8c0dd	Prevent response from sending more than once.	7 years ago
Mike Cao	7ae2011723	Sending response should not exit to allow pre-stop filters.	7 years ago
Mike Cao	1c8d06cc42	Response content-length header should use mbstring if available.	8 years ago
Mike Cao	c3c6f68909	Removed another case of $route passing. Updated tests.	8 years ago
Mike Cao	6aea7394c4	Remove $route from parameter list, only pass when dispatching callback.	8 years ago
Mike Cao	7fd3594ac7	Added ability to perform case sensitive matching of URLs.	9 years ago
Mike Cao	1feeb95ff2	Encode reserved @ character in URLs.	9 years ago
Edward Brosens	afbf306336	add getter for protected headers property	10 years ago
Seth Battin	bb11ca47bc	include pragma='no-cache' in the cache header reset	10 years ago
Mike Cao	10750b5ce3	getBody should check for PATCH method.	10 years ago
Mike Cao	a32f2c2087	getMethod should return uppercase string.	10 years ago
Mike Cao	0c8b665304	Bump version. Minor syntax update.	10 years ago
everflux	268cf0cb66	Provide content-length header if response has a body	10 years ago
Daniel Stelljes	54440d1db0	Removed static body variable from Request	10 years ago
Daniel Stelljes	703536d2bf	Set up request body caching	10 years ago
Lars-Jørgen Kristiansen	d5e808a495	fixed message for http code 306	11 years ago
Lars-Jørgen Kristiansen	b8a1c1e163	added all HTTP status codes from http://www.iana.org/assignments/http-status-codes/http-status-codes.xhtml	11 years ago
Lars-Jørgen Kristiansen	e6ed90e3fb	Added more HTTP status codes	11 years ago
Mike Cao	64d384ddff	Splat should consider trailing slash in URLs.	11 years ago
Mike Cao	88f4c1fb71	Fixed issue with splats when used with named parameters.	11 years ago
Mike Cao	7745f6a34c	Merge pull request #141 from pine3ree/patch-1 ... replacing array_push with faster []	11 years ago
maks feltrin	bcb5b120a9	Update Router.php	11 years ago
maks feltrin	229cbd24f5	replacing array_push with faster []	11 years ago
kafene	3392751dba	Add support for getting request status	11 years ago
kafene	71f66b4d5d	Add support for HTTP status code 418 (Permanent Redirect)	11 years ago
Stefan Meinecke	5e284a1d9c	Updated class comment. ... - removed deprecated json property - added missing secure, accept and proxy_ip property	11 years ago
Mike Cao	81041d418b	JSON request input is now stored in the data property.	11 years ago
Mike Cao	da40e03eb4	Removed the body property from the Request class.	11 years ago
Mike Cao	d84e51ba47	Updated route object passing functionality. ... Before the route object would automatically be passed to all callbacks. Now you need to explicitly ask for it by passing in true as the third parameter in Flight::route().	11 years ago
Mike Cao	ab2aba4e7d	Added method to get the request body.	11 years ago
Mike Cao	0f49cc7abc	Converted helper methods to public and static.	11 years ago
Cuonic	d097c21dcb	Fixed code to work when body is empty ... Now passes empty JSON data through the collection system to remove errors about unknown keys when posting via other methods	11 years ago
Cuonic	a8f50e2f4b	Fixed problem with Request Json array ... Fixed content type for browsers that also send character encoding, and now send the JSON data through the Collection system.	11 years ago
Quang Nhut Lam-Nguyen	bed3824192	Change getVar() function from private to public ... This function is very useful to get extra headers like authorization (for JSON Web Token purpose),... Flight::request()->getVar('custom');	11 years ago
Berkus Decker	c0c7b9dcd9	Modify Request based on feedback from Mike.	11 years ago
Berkus	62c4f2fa0b	Add support for parsed JSON property in Request.	11 years ago
Mike Cao	7817255a61	Added ability to get splat from URL pattern.	11 years ago
Mike Cao	30195237e5	Reverted response handling. ... Caused too many conflicts with other methods. Clean up should be done before the stop method is called.	11 years ago
Mike Cao	6a1029a187	Response should only be sent once.	11 years ago
Mike Cao	b8026d9828	Sending a response should not exit the program. ... We need to allow events to fire after the stop method is called. This will allow users to clean up resources, perform logging, etc.	11 years ago
Mike Cao	5ba910981f	Allow HTTP method overriding.	11 years ago
Mike Cao	3fde3685c5	Formatting.	11 years ago
Eric Anderson	1a0eede079	Use $_SERVER instead of getenv. ... This partially reverts cec890c. In addition to other cleanup the commit changed from using $_SERVER to getenv. As @kafene said on issue #21, getenv provides a nicer API making the code clean. I.E. $val = getenv('val') ?: 'default'; Instead of: $val = isset($_SERVER['val']) ? $_SERVER['val'] : 'default'; Although this is true unfortuantly it is less reliable. Using getenv in this way assumes the application is running under some sort of CGI interface or an interface that is simulating the CGI interface. While this assumption is very often true it is not always true. For example when using [PHP built-in webserver][1] (useful for development) getenv will return false for most of these values even through $_SERVER is populated. I believe also some interfaces on Windows don't populate getenv. This means that flight is not usable in those environments. By partially reverting cec890c we re-enable flight in those environments. For Engine.php and Response.php I just directly reverted cec890c. Since request makes so much use of getenv it seemed wise to abstract the conditional checking to keep the code clean via a private function. The tests were also updated to populate $_SERVER instead of the environment via putenv. [1]: http://php.net/manual/en/features.commandline.webserver.php	11 years ago
Mike Cao	8ca9b2d76d	Setting status should not send headers.	12 years ago
Mike Cao	2d6a04d514	Made default properties into class variables.	12 years ago
Mike Cao	d41795cbdc	Updated handling of subdirectories and redirects.	12 years ago
Mike Cao	93d7602b20	Removed caching from halt method.	12 years ago
Mike Cao	b4a9d6f70e	Fixed issue with 304 caching. Fixed PHP 5.3 compatibility.	12 years ago
Mike Cao	84078158c3	Introduced new Engine class. ... Core functionality has been moved to a namespaced Engine class. The existing Flight class is now just a static pass-through to the Engine class. Also fixed autoloading and initialization issues.	12 years ago
Mike Cao	a04020d50e	Fix line feeds.	12 years ago
Mike Cao	f64774881f	Fixed issue with url pattern matching.	12 years ago
Mike Cao	28ae5e0aec	Added route passing functionality.	12 years ago
kafene	69e5dab546	Don't let header fields clobber eachother ... See: http://php.net/manual/en/function.header.php 2nd option, $replace, should be false if sending multiple headers with the same name. The difference can be observed by editing index.php (the example index page) so that before `echo 'hello'` is a line: `flight::response()->cache(false);` and editing Response.php so that before `exit($this->body)` is a line: `var_dump(headers_list());` Before change: `array (size=3)` `0 => string 'Expires: Mon, 26 Jul 1997 05:00:00 GMT' (length=38)` `1 => string 'Cache-Control: max-age=0' (length=24)` `2 => string 'Pragma: no-cache' (length=16)` After: `array (size=5)` `0 => string 'Expires: Mon, 26 Jul 1997 05:00:00 GMT' (length=38)` `1 => string 'Cache-Control: no-store, no-cache, must-revalidate' (length=50)` `2 => string 'Cache-Control: post-check=0, pre-check=0' (length=40)` `3 => string 'Cache-Control: max-age=0' (length=24)` `4 => string 'Pragma: no-cache' (length=16)`	12 years ago
Mike Cao	4cf069a552	Minor code changes.	12 years ago
Mike Cao	65807a1f29	Parse optional parameters before regex.	12 years ago
Mike Cao	1ba8770da7	Added matched regex property.	12 years ago
Mike Cao	61ce37ca64	Updated router pattern matching.	12 years ago
Mike Cao	98d57a82b4	$_GET should not be overwritten.	12 years ago
Mike Cao	440cca5901	Minor code changes	12 years ago
kafene	96c25ff1f3	Fixed never-returned proxy IP	12 years ago
kafene	5270c6b23a	a few enhancements to request.php ... - added "secure" parameter to avoid boilerplate "if https && https != 'off'" - added "accept" parameter containing request's HTTP_ACCEPT value if any - Changed IP to always be REMOTE_ADDR, as this is the only "real" ip of the request. - Added 'proxy' parameter containing any proxy-forwarded IP - Changed getIpAddress to getProxyIpAddress - renamed $vars to $forwarded to be a bit more explicit - Removed REMOTE_ADDR from proxy forwarded addresses to detect - change explode() to sscanf, will explain this later. - added flags to the filter_var, will also explain later - added getMethodOverride function to detect request method overrides I did some research because I was wondering why it would be necessary to check for a comma, and apparently some proxies do send a comma separated list of IP addresses, with the originating client IP as the first one. It had a feeling that instead of using explode() where the first value would always be returned, there must be another way to get the first value of a token-delimited string, or just the whole string if there was no token, and I bumped in to my old friend sscanf. So the loop is now 3 levels. As far as the flags, see: http://php.net/manual/en/filter.filters.flags.php These ensure that any IP detected is not a useless IP behind a remote NAT. For example, a corporate proxy might send HTTP_X_FORWARDED_FOR using the internal IP of the user, but the only useful IP in this case is that of the proxy, which PHP gets as REMOTE_ADDR. If you find any of these changes acceptable but don't want to merge them all, do feel free to implement whichever you like. I was thinking of adding a getRequestMethod() function to check for `$_SERVER['HTTP_X_HTTP_METHOD_OVERRIDE']` or `$_POST['_method']` to allow clients to negotiate PUT/DELETE requests, but personally I am indifferent to these, as I never use methods beyond GET and POST (I'm not very RESTful I guess), and it's not really my place to be writing an implementation of either. Other frameworks seem to just overwrite the method parameter so that it appears to be the overridden one.	12 years ago
Mike Cao	f2bd6f4e02	Allow framework to be reinitialized	12 years ago