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

Invalid update manifest or mismatching applicationId #144

Open
qq253498229 opened this issue Jan 13, 2025 · 2 comments
Open

Invalid update manifest or mismatching applicationId #144

qq253498229 opened this issue Jan 13, 2025 · 2 comments

Comments

@qq253498229
Copy link

While debugging the automatic update API, an exception was thrown.

I checked all the configurations. Did I miss something?

    let url = 'https://qq253498229.github.io/demo-neu1/updater.json';
    updater.checkForUpdates(url).then(r => {
      console.log('checkForUpdates r', r);
    }).catch(e => {
      console.log('checkForUpdates e', e);
    });

manifest.json:

{
  "applicationId": "cn.codeforfun.demo-neu1",
  "version": "0.0.2",
  "resourcesURL": "http://example.com/updater_test/update_res.neu",
  "data": {
    "releaseNotes": "This is a new version with bug fixes and improvements."
  }
}

neutralino.config.json

//xxxxxxx
  "applicationId": "cn.codeforfun.demo-neu1",
  "version": "0.0.1",
  "nativeAllowList": [
    "filesystem.writeBinaryFile",
    "updater.*",
    "app.*",
    "filesystem.readDirectory"
  ],
//xxxxxx
image image
@qq253498229
Copy link
Author

I searched and reviewed some Neutralino applications on GitHub. After a comprehensive comparison, I found that the applicationId can only contain one "." and the use of "-" is not allowed.

After I changed the applicationId to "cn_codeforfun_demo_neu1", the updater.checkForUpdates function worked properly.

Is there any way to bypass this limitation?

@shalithasuranga
Copy link
Member

Hello, @qq253498229, thanks for reporting this issue and posting your findings. Please keep this issue open till we apply a fix soon with the next release as there is no workaround to bypass this regex check: https://github.com/neutralinojs/neutralinojs/blob/aafcb6afd46077e9716453e0c3fbd290ad9b1eb9/settings.cpp#L125

Thanks 🎉

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

2 participants