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

[Bug] Application crashes without internet access #113

Closed
LeeC2202Git opened this issue Jan 3, 2023 · 4 comments
Closed

[Bug] Application crashes without internet access #113

LeeC2202Git opened this issue Jan 3, 2023 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@LeeC2202Git
Copy link

Description

Application crashes when it has no internet access.

Reproduce

Steps to reproduce the behavior:
Run the application with no internet access

Expected Behaviour

As there is no listed requirement for internet access, then I simply expected the application to run without error.

If there is a requirement for internet access, then several things need to be added to comply with privacy concerns.

  1. A statement explicitly stating that internet access is required.
  2. The reason that internet access is required.
  3. Details about what information is collected and transmitted both in and out of the machine with regards to this internet connection.
  4. An option to disable the internet requirement.

Additional Context

Add any other context about the problem here or at least complete the following information.

@LeeC2202Git LeeC2202Git added the bug Something isn't working label Jan 3, 2023
@cengelha
Copy link
Member

cengelha commented Jan 3, 2023

I never experienced an issue w/o internet access but looks like the updater makes some trouble (despite it doesn't work atm). It's currently the only feature that requires internet access but shouldn't cause a crash... (there is a feature flag system as well tho, but that's not a real feature)

An overview about used online services and what data are send/received is already in the works and will be part of the wiki that will be published here on GitHub in the wiki section soon.

Giving you more control over the behavior of features requiring internet access is already on my list.

@Pd69bq
Copy link

Pd69bq commented Jan 4, 2023

experienced the same issue several times, and found similar entries in the crash report. does NomNom really needs to access api.github.com every time it starts? tbh, this "secretly" added requirement feels pretty bad.

Exception

Type

System.Net.Http.HttpRequestException

Message

A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond. (github.com:443)

@cengelha
Copy link
Member

cengelha commented Jan 4, 2023

Hey, as I mentioned, it's the built-in update feature. It checks whether a new version is available at every start (and the required information and program files are hosted here on GitHub). This not different than in the goatfungus editor nor new or "secretly" added. It's like this since the 3rd alpha almost 3 years ago and mentioned in the changelog.

And as I also mentioned in the post before, all online services will soon be explained in detail in the upcoming wiki and giving you more control in NomNom itself is already on the todo list as well.

@LeeC2202Git
Copy link
Author

I never experienced an issue w/o internet access but looks like the updater makes some trouble (despite it doesn't work atm). It's currently the only feature that requires internet access but shouldn't cause a crash... (there is a feature flag system as well tho, but that's not a real feature)

An overview about used online services and what data are send/received is already in the works and will be part of the wiki that will be published here on GitHub in the wiki section soon.

Giving you more control over the behavior of features requiring internet access is already on my list.

No problem. Out of habit, all downloaded exe files are blocked from the internet on my PC, it avoids me having to packet sniff to see what is happening in the communication chain. I also downloaded NMSCoordinates at the same time as this and that shows the same lack of internet connection in the onscreen log window but carries on running without error.

Most people don't run things without an internet connection so crashes like this are easily missed. Even though this might only be an update check, I prefer to have full control over that checking. As a developer with decades-worth of projects on my PCs, I don't leave anything to chance with unknown exe files and online communication.

@cengelha cengelha added this to the Uno milestone Jun 25, 2023
@cengelha cengelha self-assigned this Sep 11, 2023
@cengelha cengelha removed this from the Uno milestone Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants