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.

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

Add the base of http requests #78

Merged
merged 4 commits into from
Aug 30, 2021
Merged

Add the base of http requests #78

merged 4 commits into from
Aug 30, 2021

Conversation

sansyrox
Copy link
Member

@sansyrox sansyrox commented Aug 23, 2021

This pr adds the base of http compliance. Need to add more specifications per route!

https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods

@sansyrox
Copy link
Member Author

@JackThomson2 @messense , I tried adding directory serving in actix here(https://github.com/sansyrox/robyn/pull/78/files#diff-48d27698cc708c7efe770fd897e4885efaa0a15cae30d54936068603ba03bbd9R71) . However, it is not allowing me serve request apart from GET.

Do you folks know where I might be going wrong?

@sansyrox sansyrox force-pushed the http_request_finalise branch 3 times, most recently from 06aef66 to 29163d1 Compare August 29, 2021 08:46
@sansyrox sansyrox merged commit 14520b3 into main Aug 30, 2021
@sansyrox sansyrox deleted the http_request_finalise branch August 30, 2021 21:34
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.

1 participant