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

Server crashes when camera pack uses invalid characters for YMAL in the name #123

Closed
jfgilliam opened this issue Sep 30, 2017 · 0 comments

Comments

@jfgilliam
Copy link
Contributor

commented Sep 30, 2017

Need to parse the YMAL for GroupName in the cameras and quote it before the parser gets it. Also, create a option for the CameraSelector as to which camera number to start at. Then create a standard one that starts with camera 21 so the custom camera packs will show up.

Thanks to Ricky Thompson for reporting this.

@jfgilliam jfgilliam changed the title Camera Server bombs when camera pack used with invalid characters for YMAL Sep 30, 2017

@jfgilliam jfgilliam self-assigned this Sep 30, 2017

@jfgilliam jfgilliam added this to the Version 1.5 milestone Sep 30, 2017

@jfgilliam jfgilliam changed the title Server bombs when camera pack used with invalid characters for YMAL Server bombs when camera pack uses invalid characters for YMAL in the name Sep 30, 2017

jfgilliam added a commit to SIMRacingApps/SIMRacingAppsSIMPluginiRacing that referenced this issue Oct 31, 2017

Fixed #123 (SIMRacingApps/SIMRacingApps#123)
       Server bombs when camera pack uses invalid characters for YMAL in
the name

jfgilliam added a commit to SIMRacingApps/SIMRacingAppsServer that referenced this issue Oct 31, 2017

Fixed #123 (SIMRacingApps/SIMRacingApps#123)
       Server bombs when camera pack uses invalid characters for YMAL in
the name

@jfgilliam jfgilliam changed the title Server bombs when camera pack uses invalid characters for YMAL in the name Server crashes when camera pack uses invalid characters for YMAL in the name Nov 4, 2017

@jfgilliam jfgilliam closed this in ccc403c Nov 9, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.