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

Last improvements #40

Closed
wants to merge 33 commits into from
Closed

Last improvements #40

wants to merge 33 commits into from

Conversation

lucienvb
Copy link
Collaborator

  • on throw exception in cgi's child process
  • cgi delete is disabled, the normal HTTP delete will start up

Lucien Van Bussel and others added 30 commits November 9, 2023 15:20
draft CGI but need to split it up and implement with poll
c1c6ce3
added comments in handle connection for structure cgi
04305c5
took some steps in the right direction
9a80ae9
cgi send and receive almost there
3f21c8b
parseURIForCGI is finished and did some things in CGI cpp as well and…
5746c4e 
… added some comments
update parseURIForCGI
af2315f
added getters for the parameters of execve (startCGI), also made a se…
a690ab7 
…tter for CGI boolean
added getBodyLength method to HTTPRequest class
35a723c
server back running but the file manager is giving problems
cce7485
managePost now functional with the brains of sala
9321a8b
some improvements on the CGI, currently stuck on parseURIForCGI metho…
cf2e772 
…d of HTTPRequest class
stuck at the executable but big steps are taken
61c2e95
some improvents, new focus: CGI::receive
f38b390
improvents cgi
21759b6
python file and get_cgi text file
66077eb
added fcntl to GCI start
ba44ce6
fixed findClientByFd, next step is using this in handleExistingConnec…
422ec6d 
…tion and get the Client class with in his own handle connection
added getters for serverToCgiFd and cgiToServerFd and now giving Clie…
bdd718a 
…nt class to the CGI class
now using the pipes from the client in the cgi, it compiles so commit
513717a
it seems to work, need to look into the filemanager because there is …
c828151 
…something going wrong
added FileManager::manageCgi and improved CGI::receive
45e89f7
cgi works!
6c771c7
now at CGI::write
3c627be
small adjustments
6f68d8d
the cgi is f*cking working!
0b43658
a lot of cleaning up in CGI.cpp
cad2502
did some cleaning up for the client.cpp
1e1a3e2
did some cleaning up on HTTPServer.cpp
d325322
some additions and cleaning up
5164449
post_cgi also working with additional path and query_string, and got …
5070531 
…rid of garbage when sending in the cgi, also replaced some normal exeptions for ClientExceptions
@lucienvb lucienvb closed this Dec 13, 2023
@lucienvb
Copy link
Collaborator Author

had a stupid copy pasta mistake in there

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@lucienvb