Skip to content

Add CGI support#22

Closed
miky-rola wants to merge 7 commits intoAz107:masterfrom
miky-rola:new-feature/cgi
Closed

Add CGI support#22
miky-rola wants to merge 7 commits intoAz107:masterfrom
miky-rola:new-feature/cgi

Conversation

@miky-rola
Copy link
Copy Markdown
Contributor

  • CGI Support: Execute dynamic scripts via the Common Gateway Interface.

@Az107
Copy link
Copy Markdown
Owner

Az107 commented Apr 9, 2025

The base branch is outdated, we need to rebase it against master to bring in the latest changes.
Also, your CGI module isn’t being used anywhere; the code is still relying on my prototype function in main.rs.
Maybe we could unify everything into the cgi module.
Also, it might make more sense for the cgi module to be part of the standalone server instead of the core library?
If you need any help with the rebase or reorganizing things, feel free to ask!

@Az107 Az107 self-requested a review April 9, 2025 22:55
@miky-rola
Copy link
Copy Markdown
Contributor Author

miky-rola commented Apr 10, 2025

I'll need help with the rebasing, but I'll implement the suggestions regarding the cgi

@Az107
Copy link
Copy Markdown
Owner

Az107 commented Apr 10, 2025

No problem , I updated my branch feature/cgi
because had a lot of conflicts.
For future reference you could run
git rebase master
and it will update your checked branch against master, this could bring some conflicts so you need to fix these manually.

@Az107 Az107 closed this Apr 11, 2025
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.

2 participants