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 a GET route for validate tunnel so we can use caddy on_demand #405

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

SiebeVE
Copy link
Contributor

@SiebeVE SiebeVE commented Mar 13, 2024

Copy link

what-the-diff bot commented Mar 13, 2024

PR Summary

  • Addition to .gitignore
    We've made improvements to protect sensitive details by adding db.db to the .gitignore file. This ensures that our database doesn't inadvertently get exposed in the public code.
  • Updated Server Factory
    The app/Server/Factory.php file has been updated to handle a new route (/validate-tunnel). An additional mechanism, addValidateTunnel(), has been added to facilitate this new routing.
  • New File: ValidateTunnelController
    A new file app/Server/Http/Controllers/ValidateTunnelController.php has been introduced. This forms part of our efforts to manage new requests efficiently. It specifically deals with the handling of requests to /validate-tunnel.
  • Updates in configuration
    We've included a new configuration for validate_tunnel in the config/expose.php file. It accommodates an authorized key, therefore enhancing our security measures.

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