n42n / n3n

Peer to Peer VPN
51 stars 6 forks source link

JsonRPC support replaces the old API #11

Closed hamishcoleman closed 4 months ago

hamishcoleman commented 4 months ago

Some of the tools still need porting to the new API, but all server features are present, working and documented. There is still plenty of room for improvements, but this branch contains the framework and expected minimum features.

One nice feature is that there is now a built in HTTP webpage that allows tool-free access to the running status on any system where there is a web browser (specifically, Windows, where you cannot expect people to run a python script)

Also:

codecov-commenter commented 4 months ago

Codecov Report

Attention: 591 lines in your changes are missing coverage. Please review.

Comparison is base (5249d46) 43.79% compared to head (a803884) 38.23%.

Files Patch % Lines
src/management.c 31.95% 247 Missing :warning:
src/conffile.c 10.71% 100 Missing :warning:
src/sn_utils.c 43.51% 74 Missing :warning:
src/edge_utils.c 64.49% 49 Missing :warning:
src/resolve.c 53.19% 44 Missing :warning:
apps/supernode.c 0.00% 31 Missing :warning:
apps/edge.c 56.52% 20 Missing :warning:
src/base64.c 61.76% 13 Missing :warning:
src/sn_selection.c 14.28% 6 Missing :warning:
src/auth.c 0.00% 2 Missing :warning:
... and 4 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #11 +/- ## ========================================== - Coverage 43.79% 38.23% -5.57% ========================================== Files 38 51 +13 Lines 5754 8898 +3144 ========================================== + Hits 2520 3402 +882 - Misses 3234 5496 +2262 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.