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

fix: Move online-shell dev env to /online documentBase - #242 #243

Merged
merged 1 commit into from
Dec 6, 2023

Conversation

phantomjinx
Copy link
Member

  • Aligns the online-shell dev server with the production build by making the documentBase of the app /online.

  • Avoids the problem, reported in Can no longer connect to jolokia urls in online-shell dev server #242, of an invalid double-slash url since the documentBase is identified as /online.

  • hawtconfig.json

    • Corrects urls for hawtio-logo not showing up correctly in dev with documentBase but still showing up correctly in production
  • webpack.config.common.js

    • Does not effect production but make publicPath always /online regardless of mode
  • webpack.config.dev.js

    • Modifies output and static to include the publicPath
    • Updates the osconsole url which is the only path relative in the code base so needs the publicPath prepended

* Aligns the online-shell dev server with the production build by making
  the documentBase of the app /online.

* Avoids the problem, reported in hawtio#242, of an invalid double-slash url
  since the documentBase is identified as /online.

* hawtconfig.json
  * Corrects urls for hawtio-logo not showing up correctly in dev with
    documentBase but still showing up correctly in production

* webpack.config.common.js
  * Does not effect production but make publicPath always /online
    regardless of mode

* webpack.config.dev.js
  * Modifies output and static to include the publicPath
  * Updates the osconsole url which is the only path relative in the code
    base so needs the publicPath prepended
@phantomjinx phantomjinx self-assigned this Dec 5, 2023
@phantomjinx phantomjinx merged commit 8688f4b into hawtio:main Dec 6, 2023
3 checks passed
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.

2 participants