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

OpenVPN rock-on install wizard fails to show pre-install summary #2913

Open
phillxnet opened this issue Oct 4, 2024 · 3 comments
Open

OpenVPN rock-on install wizard fails to show pre-install summary #2913

phillxnet opened this issue Oct 4, 2024 · 3 comments
Labels
UI-enhancement Enhancement, renovation, update within Web-UI.

Comments

@phillxnet
Copy link
Member

While developing against "Rockon pre & post-install summaries are inconsistent" #2904 an OpenVPN install wizard failed to show the pre-install config summary page: a second Confimation on the existing dialog continued the installation which succeeded successfully.

With debug enabled the following log resulted when installing this Rock-on: the only Rock-on remaining that uses the "custom_config" key: passed from front-end to backend in the following as cc_map

[04/Oct/2024 18:16:54] DEBUG [storageadmin.views.rockon_id:108] install request with share_map={}, port_map={'1194': 1194}, dev_map={}, cc_map={'servername': 'fqdn.fake.biz'
}, env_map={}

Browser console error:

Uncaught TypeError: can't convert undefined to object

with a reference to our storageadmin.js (all-in-one js lib) re:

nextPage: function() {
        var _this = this;
        var promise = !_.isNull(this.currentPage) ?
            this.currentPage.save() :
            $.Deferred().resolve();
        promise.done(function(result, status, jqXHR) {
            _this.incrementPage();
        });
        promise.fail(function(jqXHR, status, error) {
            console.log(error);
        });
    },

Post install the entered details appear as expected (post proposed changes associated with #2904) in the post install config summary:

Resource type Internal Reference External Reference / Configured value
Port 1194 1194
Custom servername fqdn.fake.biz
@Hooverdan96
Copy link
Member

May be it's also time to retire this docker image and switch to the offical one that's available (don't know when it was first posted) from openvpn, and possibly retire the final custom_config usage?

https://hub.docker.com/r/openvpn/openvpn-as

Has to be tested, but it seems that it's pretty self-contained and also contains an ui component.

@phillxnet
Copy link
Member Author

@Hooverdan96 Agreed on all accounts. I'd not looked into the availability of an official image, and so if there is one available then it definitely our current offering - with the custom config and lots of command-line requirements. The Official one can't be much worse :) .

... and also contains an ui component.

OK, so that's a major win. Nice find. Yes, lets drop this Rock-on entirely and go in that direction.

@Hooverdan96
Copy link
Member

Hooverdan96 commented Oct 9, 2024

I opened #392 on the rockon-registry repo for this.

@phillxnet phillxnet added the UI-enhancement Enhancement, renovation, update within Web-UI. label Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI-enhancement Enhancement, renovation, update within Web-UI.
Projects
None yet
Development

No branches or pull requests

2 participants