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

Replace www-path with www-proxy option for development #248

Merged
merged 9 commits into from
Sep 26, 2021

Conversation

anbraten
Copy link
Member

@anbraten anbraten commented Jul 19, 2021

By adding a new ENV variable called WOODPECKER_WWW_PROXY it is possible to serve a webinterface via a proxy configured by the WOODPECKER_WWW_PROXY value for development instead of serving the interface from the bundled code or from some folder location as the old WOODPECKER_WWW option allowed. Using a proxy allows developing the UI with hot-reloading.

image

cmd/drone-server/server.go Outdated Show resolved Hide resolved
@6543 6543 added the feature add new functionality label Jul 19, 2021
@6543 6543 added this to the 0.15.0 milestone Jul 19, 2021
@6543
Copy link
Member

6543 commented Jul 19, 2021

I'll have a closer look & test it once v0.14.0 got released - witch is only hold back by one issue

@6543

This comment has been minimized.

@anbraten

This comment has been minimized.

@anbraten anbraten mentioned this pull request Jul 31, 2021
58 tasks
router/router.go Outdated Show resolved Hide resolved
cmd/drone-server/server.go Outdated Show resolved Hide resolved
@anbraten anbraten changed the title feat: add www-proxy for development Replace www-path with www-proxy option for development Sep 18, 2021
@anbraten anbraten requested a review from 6543 September 26, 2021 19:56
@6543 6543 merged commit bd19f90 into woodpecker-ci:master Sep 26, 2021
@anbraten anbraten deleted the proxy-dev-server branch September 26, 2021 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature add new functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants