Added webinterface option for C++ chat client, small Rust config changes

[kieranjazy]

Added C++ web interface available by default. Can be disabled by removing --webinterface from Enarx.toml. Uses port 8080. Made small changes to Rust files to fix broken file locations

[rjzak]

A better way to do things might have been to rebase #69 and update.

• So you make your changes.
• git commit -as Commit you new changes
• git rebase -i HEAD~2 Compare last two commits
• Replace the pick with f
• Save, and git will "fix-up" the commit with the next commit, keeping the history clean.
• git push -f origin <my branch> to force push, and if you have an open PR, it'll be automatically updated.

Also, you should probably work in a new branch instead of main. This is easy:

• git checkout main
• git pull origin main to get any changes which might be in your repo
• git remote add upstream https://github.com/enarx/codex.git add the main repo as the "upstream"
• git fetch upstream get the upstream history
• git rebase upstream/main get the latest changes from upstream, add to your main
• git push origin main freshen up your copy of main
• Make some changes to your code, don't commit yet!
• git checkout -b <my_new_branch>
• git push origin <my_new_branch>
• Open PR against your branch
• Updates (shown above) will automatically update your PR, don't need to close and make new ones!

If you have any questions, let me know, or ask in the Enarx chat.

A tutorial: https://www.atlassian.com/git/tutorials/rewriting-history/git-rebase
An example: notice all the force push messages enarx/steward#70

[kieranjazy]

Would it be OK to merge this PR as it is now or should I add it all to a new branch? Was trying to squash all the commits into one but didn't get it right

[rjzak]

Don't worry about having a new branch this time.

It seems one of the checks is failing because assets.h can't be found, perhaps @rvolosatovs will be able to assist.

[kieranjazy]

I've it fixed on local, was using your python script to generate the webpage assets but didn't realise it'd break something so I've included it. I'll upload final commit now and solve merge issues

[rjzak]

That's fine for now, but the Nix code should be updated to generate assets.h instead of including it.

[kieranjazy]

That's fine for now, but the Nix code should be updated to generate assets.h instead of including it.

Ah ok, I'll look into Nix, haven't used it before

[nickvidal]

Hi @kieranjazy, I tested your PR and it works great. Thanks!

[kieranjazy]

Thanks Nick, just struggling to get Nix to run a Python script for me, still working on it

[rvolosatovs]

That's fine for now, but the Nix code should be updated to generate assets.h instead of including it.

I don't think we need that, at least it should not be a blocker for this PR, this warrants more discussion. Can we file a "goldplating" tech debt issue about it and get this merged?

[rvolosatovs]

@kieranjazy I believe the outdated wasi-libc issue is now solved, but the code still fails to compile with:

chat-client-wasm32-unknown-wasi> /nix/store/fxasr4jvncasd20wkisj6wf78gyww84y-c++/main.cpp:17:21: warning: unused variable 'HTTP_NOT_FOUND' [-Wunused-const-variable]
chat-client-wasm32-unknown-wasi> const unsigned char HTTP_NOT_FOUND[14] = "HTTP/1.0 404\n";
chat-client-wasm32-unknown-wasi>                     ^
chat-client-wasm32-unknown-wasi> /nix/store/fxasr4jvncasd20wkisj6wf78gyww84y-c++/main.cpp:19:21: warning: unused variable 'CONTENT_TYPE_PLAIN' [-Wunused-const-variable]
chat-client-wasm32-unknown-wasi> const unsigned char CONTENT_TYPE_PLAIN[27] = "Content-Type: text/plain\n\n";
chat-client-wasm32-unknown-wasi>                     ^
chat-client-wasm32-unknown-wasi> 2 warnings generated.
chat-client-wasm32-unknown-wasi> /build/.attr-0l2nkwhif96f51f4amnlf414lhl4rv9vh8iffyp431v6s28gsr90: line 3: -o: command not found