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

Develop Log #2

Closed
timzaak opened this issue Jun 4, 2024 · 2 comments
Closed

Develop Log #2

timzaak opened this issue Jun 4, 2024 · 2 comments

Comments

@timzaak
Copy link
Collaborator

timzaak commented Jun 4, 2024

V2.1.1 to fix

    • SPA-Multiple File should create before upload, write sub path when release.
    • delete old version cache when new version files is on serving. old version: serving version - 3
    • server cold start should serving old templates. old version: serving version -3

V2.2.0

Fix

    • acme feature support check passed (need public IP)
    • admin server should get all errors when https/http init. (need refact admin server)
    • fix delete domain verison while cacheFile does not delete.

Test

    • https self signed certificate check passed.
    • delete domain verison check passed.
    • cache expired check passed.

Doc

    • break change http config
    • add acme doc

Before V2.3.0

Test

    • acme feature support check passed (need public IP)
    • check js client handle interrupt upload process if correctly
    • check file cache if handle cache correctly

Improve

    • js client sdk error info improve (multiple domain)
    • server config load can be injected for easy test and test parallam.
    • multiple SPA in one domain support index.html redirect.
    • domain config support sub dirs. (this needs to rewrite DomainConfig init and get with default method)
    • add get acme cert info API

Before V2.4.0

    • split common entity between server and client to other crate, decrease client size.
    • gracefully shutdown
@timzaak timzaak closed this as completed Jun 7, 2024
@timzaak timzaak reopened this Jun 17, 2024
@timzaak
Copy link
Collaborator Author

timzaak commented Jun 18, 2024

CORS

if port is not 80、443, Browser would set Origin header which would treat as CORS request.

  1. delete cors check for METHOD::GET request
  2. tell user to add cors=true config

which would be better?

@timzaak
Copy link
Collaborator Author

timzaak commented Jun 27, 2024

Domain Alias

use http header status MOVED 301 to redirect.
Https needs lots of things to do.

    • ACME, needs to request multiple domain, and needs to register one cert to multiple domain(init and update) .
    • admin server: http request redirect to http alias, when has http_redirect_https_port config, then needs to rewrite location to the alias https.
    • admin server: upload files /get file postion API should forbid uploading with alias name.
    • add integration tests.
    • Https with single domain cert would have problems. need notify user. more config check.

http has three status: open, disable, redirect to https, default is open/disable by if http port binded.
serve bebind proxys, needs external port.

@timzaak timzaak changed the title bugs to remember Develop Log Jun 27, 2024
@timzaak timzaak closed this as completed Jul 1, 2024
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

No branches or pull requests

1 participant