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

Plugin registers task while disabled #1397

Open
1 task done
xknat opened this issue Nov 3, 2023 · 5 comments
Open
1 task done

Plugin registers task while disabled #1397

xknat opened this issue Nov 3, 2023 · 5 comments
Assignees
Labels
type: 🐛 bug A confirmed bug in the plugin which needs to get resolved

Comments

@xknat
Copy link
Member

xknat commented Nov 3, 2023

Is there an existing feature request for this?

  • I have searched the existing feature requests

Describe your feature request

image

@xknat xknat added type: 💡 enhancement New feature request / improvement of an existing feature type: 🐛 bug A confirmed bug in the plugin which needs to get resolved and removed type: 💡 enhancement New feature request / improvement of an existing feature labels Nov 3, 2023
@CSneko
Copy link

CSneko commented Nov 5, 2023

I also meet the same problem,look like it's can't check update form github.
In this class:UpdateCheckerGitHub.checkForUpdate(),

            GitHubReleaseInfo releaseInfo = response.getBodyAs(GitHubReleaseInfo.class);

            Optional<GitHubAssetInfo> jarAsset = releaseInfo.getAssets().stream()
                    .filter(asset -> asset.getName().equals(JAR_ASSET_NAME))
                    .findFirst();

getAssets() retrun null

@CSneko
Copy link

CSneko commented Nov 7, 2023

I found a solution:
Clone source,then open net.skinsrestorer.shared.update.UpdateCheckerGitHub
set value of private static final String RELEASES_URL_LATEST,look like:

private static final String RELEASES_URL_LATEST = "https://w.csk.asia/other/latest.json";

Then building,It's OK.
tips:
It will affect update functionality!
You can replace https://w.csk.asia/other/latest.json to your website.

@AlexProgrammerDE
Copy link
Member

This basically just changes the update endpoint. The proper fix needs to be found, not by using a third party API, which will not be up-to-date with the actual latest version in the long run.

@AlexProgrammerDE
Copy link
Member

Using a third-party endpoint is gonna give the domain host the ability to install malicious jars onto your server just by modifying the jar download url.

@CSneko
Copy link

CSneko commented Nov 7, 2023

Using a third-party endpoint is gonna give the domain host the ability to install malicious jars onto your server just by modifying the jar download url.

Yes,You are right.What I did was just solve the error.What can be seen is that the country where his server is located may not be able to connect properly with github.
I don’t know if this error report has other effects.So I just get the json file form github api,then upload to my website.
https://w.csk.asia/ is my plugins' page.I upload somthing for my server because my server also can't connect to github.com.
The correct approach (in my opinion) is to build a simple website that can be accessed in his country/region to obtain the github api, (for example: Cloudflare Workers).But it's maybe difficult.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: 🐛 bug A confirmed bug in the plugin which needs to get resolved
Projects
Status: 🆕 New
Development

No branches or pull requests

3 participants