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

Killswitches are no longer found #2252

Closed
2 tasks done
chennin opened this issue Jun 6, 2024 · 5 comments
Closed
2 tasks done

Killswitches are no longer found #2252

chennin opened this issue Jun 6, 2024 · 5 comments
Assignees
Labels
bug unconfirmed An unconfirmed bug

Comments

@chennin
Copy link
Contributor

chennin commented Jun 6, 2024

Please complete the following information:

  • Version: Running from source: E:D Market Connector 5.11.0+f93485ec.DIRTY
  • Game Version: -
  • OS: Arch Linux
  • OS Locale: English
  • If applicable: -

Describe the bug

Killswitches seem to have disappeared. The log contains the following, and both killswitch URLs are currently 404ing

2024-06-06 16:45:37.155 UTC - INFO - 66301:126246759634752:66301 killswitch.fetch_kill_switches:346: Attempting to fetch kill switches
2024-06-06 16:45:37.232 UTC - WARNING - 66301:126246759634752:66301 killswitch.fetch_kill_switches:361: Failed to get kill switches, data was invalid: Extra data: line 1 column 4 (char 3)
2024-06-06 16:45:37.233 UTC - WARNING - 66301:126246759634752:66301 killswitch.get_kill_switches:447: could not get killswitches, trying fallback
2024-06-06 16:45:37.234 UTC - INFO - 66301:126246759634752:66301 killswitch.fetch_kill_switches:346: Attempting to fetch kill switches
2024-06-06 16:45:37.315 UTC - WARNING - 66301:126246759634752:66301 killswitch.fetch_kill_switches:361: Failed to get kill switches, data was invalid: Extra data: line 1 column 4 (char 3)
2024-06-06 16:45:37.316 UTC - WARNING - 66301:126246759634752:66301 killswitch.get_kill_switches:451: Could not get killswitches.

OLD_KILLSWITCH_URL = 'https://raw.githubusercontent.com/EDCD/EDMarketConnector/releases/killswitches.json'
DEFAULT_KILLSWITCH_URL = 'https://raw.githubusercontent.com/EDCD/EDMarketConnector/releases/killswitches_v2.json'

To Reproduce
Steps to reproduce the behavior:

Run EDMC and look at the log/output

Expected behavior

As a user, I have no idea if this is visible or not

Logs
EDMarketConnector-debug.log

@chennin chennin added bug unconfirmed An unconfirmed bug labels Jun 6, 2024
@Rixxan
Copy link
Contributor

Rixxan commented Jun 6, 2024 via email

@Athanasius
Copy link
Contributor

The weird thing is I just cannot find the commit in releases that removed the file.

git log --name-status --full-history -- killswitches_v2.json - Has the 'A' line for when it was added, no 'D' line, just a whole lot of merges.
git log --diff-filter=D --summary - Also doesn't show the file at all, but does show other file deletions.

It feels like something has become very broken with the git commit history, at least for me locally. How can a file, that clearly shows as having been added in f833afd , that has no deletion logged, not be present ?

@Rixxan
Copy link
Contributor

Rixxan commented Jun 6, 2024 via email

@Athanasius
Copy link
Contributor

Yeah, in this case no live actual killswitch was ever added, so the content just needs to be:

{
    "version": 2,
    "last_updated": "18 October 2021",
    "kill_switches": []
}

@Rixxan
Copy link
Contributor

Rixxan commented Jun 6, 2024

Re-added. Thank you for the report and catch!

@Rixxan Rixxan closed this as completed Jun 6, 2024
@Rixxan Rixxan self-assigned this Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug unconfirmed An unconfirmed bug
Projects
None yet
Development

No branches or pull requests

3 participants