FSGModding / FSG_Mod_Assistant

Farming Simulator 22 Mod Manager.
https://fsgmodding.github.io/FSG_Mod_Assistant/
ISC License
74 stars 10 forks source link

[VERSION TRACK] v3.4.1 Known Issues & Bugs #210

Closed jtsage closed 7 months ago

jtsage commented 8 months ago

Known Bugs - v3.4.1

This issue is an umbrella issue to keep track of bugs in the current version. You can reply directly here, but opening a new issue would make life easier for us.

Issue List (included in 3.4.1)

Issue List (included in 3.5.0-RC1)

Issue List (included in 3.5.0-RC2)

Issue List (included in 3.5.0-RC4)

Issue List -- Included in 3.5.0-RC6, most in #218

Issue List - Included in 3.5.0-RC7

Special Note: there is no way I didn't break a few things on this RC, at least 1 or 2 more are going to be forthcoming before 3.5.0-RELEASE.

Progress

vitalii2011 commented 7 months ago

This https://discord.com/channels/949619926463840327/1009879187429003377 is very frustrating. Image 001 ScreenShot_20240211150212-1

jtsage commented 7 months ago

working on it right now. it'll be in the next update

jtsage commented 7 months ago

I just pushed the changes for the malware detector - there are probably other false positives out there, but I did check it against the entire 'gameplay' category on modhub, along with the popular script mods that I use. Please use issue #220 to report any others. If you want a safe file to test with, see here for my test mod, it has the offending code in a LUA file that is neither loaded nor able to actually run.

2024-02-11 19_11_42-Possible Malware Detected

vitalii2011 commented 7 months ago

Users may have problems with 3.5.0-RC7!

vitalii2011 commented 7 months ago

When "I do not trust it, delete it right now" are archives with mods deleted not to the Recycle Bin/bypassing the Recycle Bin?

jtsage commented 7 months ago

When "I do not trust it, delete it right now" are archives with mods deleted not to the Recycle Bin/bypassing the Recycle Bin?

that's correct - they are removed from disk directly, it bypasses the recycle bin. In this case, I could send it to the recycle bin - keep in mind, this is detecting mods with funky scripting (there were a whole bunch released in the last few days on some sites) - it is not, nor is intended to be a system wide malware scan - there are plenty of options for that already. This is only looking for mods that may maliciously alter your farm sim installation only.

vitalii2011 commented 7 months ago

At the first launch, the color of the text did not change when hovering the cursor (the next time it changed):

https://github.com/FSGModding/FSG_Mod_Assistant/assets/41552153/396d0bfb-8acb-45d8-be9c-32069032be0e

it would be better if the arrow became a “pointing finger” - not everyone will be able to see the change in text color and understand that when clicked, an action from the text will occur.

jtsage commented 7 months ago

Oh, and some additional information on the rest of the buttons - the first one you mentioned does indeed delete the mod.

The second "I want to investigate this" adds it to an ignore list by hash and folder hash - if you move or copy the file to another collection, the warning will re-display for that copy of the mod. Also, that ignore flag is not saved, so the next time you start Mod Assitant, the warning will show again.

Finally the "I'm sure I trust this..." (last button) adds it to a saved list in your preferences, by name only - it will not show the "malware" tag, and it will not ask you about that mod ever again.

jtsage commented 7 months ago

At the first launch, the color of the text did not change when hovering the cursor (the next time it changed):

it would be better if the arrow became a “pointing finger” - not everyone will be able to see the change in text color and understand that when clicked, an action from the text will occur.

Got it - when I ran the RC build I noticed some odd behavior as well - it looked like the dialog was getting hung up in or under the changelog of the splash screen - I'll look at the timing of events on those. And i'll look at changing the pointer type as well.

vitalii2011 commented 7 months ago

For some reason, MA didn’t like FS19_GlobalCompany. And deleted (due to my carelessness) VehicleInspector VehicleInspector.zip

jtsage commented 7 months ago

whoops. I should have checked vehicleinspector - I forget what his other massive overlay one was called, but I bet that does it too. And for some reason it never occurred to me to run it against my copy of 19 mods - I bet there's a fair few in there too. Thanks, I'll get them added prior to the next release

vitalii2011 commented 7 months ago

That's why there was a question about the reason for deleting bypassing the Recycle Bin. Someone may delete something that they edited for a long time and/or cannot restore.

jtsage commented 7 months ago

That's why there was a question about the reason for deleting bypassing the Recycle Bin. Someone may delete something that they edited for a long time and/or cannot restore.

I agree - I'll change this call to send it to the recycle bin instead.

EDIT: if you want to alter the translated text on that button to reflect that, I will do the same.

vitalii2011 commented 7 months ago

It might be possible to make a list of detected/removed malicious files somewhere here: 0010 Because this: Image 003 may remain undeleted (I understood your explanation of why they are not there) without tags, etc. Pests exist not only on the Internet) For example, a child can download various cool cars with nasty things and mark them all as “acceptable”.

jtsage commented 7 months ago

yeah, I will add what you have "permanently" ignored in there, along with a button to clear it out. I was considering having that list clear itself on the "Clear Cache" button as well

vitalii2011 commented 7 months ago

EDIT: if you want to alter the translated text on that button to reflect that, I will do the same.

No. For what? If I get you right. )

jtsage commented 7 months ago

EDIT: if you want to alter the translated text on that button to reflect that, I will do the same.

No. For what? If I get you right. )

I'll probably change that button from

I do not trust it, delete it right now

to something like

I do not trust it, move it to the recycle bin it right now

vitalii2011 commented 7 months ago

whoops. I should have checked vehicleinspector - I forget what his other massive overlay one was called …

https://www.modhoster.com/community/user/happylooser#t2

jtsage commented 7 months ago

oh, cool, thanks - I've not used any of his stuff in a while - but I know he spent a ton of time on library and utility functions, so they probably all use similar code to manage saved settings which is causing the false positive on this

vitalii2011 commented 7 months ago

https://github.com/loki79uk/FS22_NoDelete/releases/tag/v0.0.0.1

vitalii2011 commented 7 months ago

https://discord.com/channels/645566659075244043/645566659075244049

jtsage commented 7 months ago

https://github.com/loki79uk/FS22_NoDelete/releases/tag/v0.0.0.1

Aye, Loki and I were talking about this yesterday a bit - looks to be a useful bit of protection.

For what it's worth, this mod was actually the second entry into the global whitelist for MA's scan, as his test case there also triggers the warning :)