Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Handle JSON post and get requests #380

Closed
wants to merge 22 commits into from
Closed

Handle JSON post and get requests #380

wants to merge 22 commits into from

Conversation

mudhoney
Copy link
Contributor

  • introducing initial psr-4 and namespacing
  • implement mechanism for handling both get and json posts requests
  • add POST, PUT, PATCH to cors methods and add content-type to allowed header list
  • implement tests
  • mark sometests as skipped and label them as integration tests

@mudhoney mudhoney requested a review from dgarciabriseno May 28, 2024 14:47
dgarciabriseno
dgarciabriseno previously approved these changes May 28, 2024
View reviewed changes
dgarciabriseno and others added 14 commits May 28, 2024 15:56
@dgarciabriseno
Better event interface (Helioviewer-Project#379)
49941c8 
* Remove postprocessor

* Pass observation time to event interface

* Fix event query time range

Events were previously queried from the given time to 1 day forward.
Now events are queried from the given time and 12 hours forward & back.

* Update composer dependencies

* Set 500 http status for errors

* Fix typo, rm debug print

* Print container logs

* always print docker logs

* Update CI

* Remove cli

* update CI logging
@mudhoney
fix conflict when rebasing
8f1784d
@dgarciabriseno @mudhoney
Print container logs
c8ccf97
@dgarciabriseno @mudhoney
always print docker logs
1a3cced
@dgarciabriseno @mudhoney
Update CI
626589a
@mudhoney
fix more conflict php.yml
c811d30
@mudhoney
set autoload namespace
a9b0b0f
@mudhoney
request params implementation and index.php to handle exceptions
ba19c8f
@mudhoney
tests for request with mockphpstream handler
1f525cd
@mudhoney
add content-type header to cors allowed headers
247b33f
@mudhoney
client states now using JSON post
5ff8bc8
@mudhoney
default cors allowed methods, and remove pr/pre function added before
bbb1a6d
@mudhoney
mark some of the tests as integration tests, will be handled later
c8f3994
@mudhoney
fix merge conflict
515e318
@mudhoney mudhoney closed this May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dgarciabriseno dgarciabriseno dgarciabriseno left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mudhoney @dgarciabriseno