Labels Mijlpalen

Nieuw probleem

Use PSR-7 #53

Closed

opened 7 jaren geleden by jking · 2 comments

jking commented 7 jaren geleden

Eigenaar

PHP-FIG published PSR-7, a model for representing HTTP requests and responses in a consistent and coherent way. Currently Arsse has its own ad hoc lightweight representation, but switching to a tested PSR-7 implementation (such as that provided by Guzzle) would probably reduce the potential for bugs.

PHP-FIG published [PSR-7](http://www.php-fig.org/psr/psr-7/), a model for representing HTTP requests and responses in a consistent and coherent way. Currently Arsse has its own ad hoc lightweight representation, but switching to a tested PSR-7 implementation (such as that provided by [Guzzle](https://github.com/guzzle/psr7)) would probably reduce the potential for bugs.

jking verwees naar dit probleem vanuit commit 6 jaren geleden'

Replace Response objects with PSR-7 response messages; improves #53 While the test suite passes, this commit yields a broken server: replacing ad hoc request objectss with PSR-7 ones is still required, as is emission of PSR-7 responses. Both will come in subsequent commits, with tests Diactoros was chosen specifically because it includes facilities for emitting responses, something which is awkward to test. The end of this refactoring should see both the Response and Request classes disappear, and the general REST class fully covered (as well as any speculative additions to AbstractHanlder).

jking verwees naar dit probleem vanuit commit 6 jaren geleden'

Replace Resquest objects with PSR-7 request messages; improves #53

jking mijlpaal bewerkt van Future 0.3.0 6 jaren geleden

jking commented 6 jaren geleden

Poster

Eigenaar

For the time being, we will be using Diactoros for PSR-7 functionality. It provides significantly more compelling features than Guzzle.

For the time being, we will be using [Diactoros](https://zendframework.github.io/zend-diactoros/overview/) for PSR-7 functionality. It provides significantly more compelling features than Guzzle.

jking verwees naar dit probleem vanuit commit 6 jaren geleden'

Initial rewrite of REST class; needs more testing, but should be functional - improves #53 - improves #66

jking verwees naar dit probleem vanuit commit 6 jaren geleden'

Tests and fixes for REST class; fixes #53; improves #66

jking heeft dit probleem gesloten 6 jaren geleden

jking commented 6 jaren geleden

Poster

Eigenaar

Authentication is not properly handled via the server requests; this needs to be done to complete functionality.

Authentication is not properly handled via the server requests; this needs to be done to complete functionality.

jking heropende dit probleem 6 jaren geleden

jking verwees naar dit probleem vanuit commit 6 jaren geleden'

Use PSR-7 for authentication; fixes #53

jking heeft dit probleem gesloten 6 jaren geleden

Log in om deel te nemen aan deze discussie.

Geen Branch/Tag gespecificeerd

Branches Labels

arch

arch2

dbtest

getopt

master

microsub

phake

reader

redup

rpm

0.1.0

0.1.1

0.10.0

0.10.1

0.10.2

0.10.3

0.10.4

0.10.5

0.2.0

0.2.1

0.3.0

0.3.1

0.4.0

0.5.0

0.5.1

0.6.0

0.6.1

0.7.0

0.7.1

0.8.0

0.8.1

0.8.2

0.8.3

0.8.4

0.8.5

0.9.0

0.9.1

0.9.2

Labels

Labels toepassen

Verwijder labels

admin tools api bug documentation duplicate enhancement feature help wanted in progress internals invalid packaging question testing trivial wontfix

Geen label admin tools api bug documentation duplicate enhancement feature help wanted in progress internals invalid packaging question testing trivial wontfix

Mijlpaal

Stel mijlpaal in

Verwijder mijlpaal

Geen items

Geen mijlpaal

0.3.0

Toegewezen aan

Gebruikers toewijzen

Verwijder toegewezen aan

Niet toegewezen

1 deelnemers

Notificaties

Abonneren

Vervaldatum

De deadline is ongeldig of buiten bereik. Gebruik het formaat 'jjjj-mm-dd'.

Geen vervaldatum ingesteld.

Afhankelijkheden

Deze kwestie heeft momenteel geen afhankelijkheden.

Schrijf Voorbeeld

Laden…

Annuleren

Opslaan

Er is nog geen inhoud.