View Issue Details

IDProjectCategoryView StatusLast Update
0008803Multi Theft Auto : San AndreasClientpublic2015-04-20 12:16
ReporterGallardo9944Assigned Toccw 
PrioritynormalSeverityfeatureReproducibilityalways
Status resolvedResolutionfixed 
Product Version 
Target Version1.5Fixed in Version1.5 
Summary0008803: [Request] Remove BASS Error message for players
Description

I'm forced to create a sound every 1-2 seconds to check if the stream has started. If it's down, player's console is spammed with BASS Error because there is no such stream.

Is there any possibility of hiding the message? Put it under setDevelopmentMode or something. I am pretty sure players shouldn't see such stuff by default.

TagsNo tags attached.

Activities

ccw

2015-03-27 07:40

administrator   ~~0023127

Have you tried onClientSoundStream, or is there a problem with it?

Gallardo9944

2015-03-27 07:48

viewer   ~~0023128

I need to check if a livestream is up to start listening to it. Apparently there's no other way than trying to connect and see if it was successful.

MrTasty

2015-03-27 21:10

viewer   ~~0023132

Last edited: 2015-03-28 12:34

View 7 revisions

Also, BASS Error messages include an URL to the file you're attempting to play, could be a high risk of unauthorised entry to your HTTP/FTP files, when attempting to play music from a password protected directory (as the URL includes username and password to the file, eg. http:// admin:password123 @mtasa.com/scripts/sound.mp3 [spaces added to avoid making a hyperlink to a 404 ^^])

Noneatme

2015-03-28 12:56

viewer   ~~0023135

Yes please, this is a security risk.

sbx320

2015-03-28 18:43

administrator   ~~0023136

Anyone can easily grab those credentials with a regular packet sniffer, so you really shouldn't rely on those to stay a secret.

Moving BASS errors to debugscript might be a good idea to avoid user confusion anyway. To fix this specific issue I'd also suggest to move the connect failure message for streams to onClientSoundStream (to avoid spamming debugscript with irrelevant information).

ruun

2015-03-31 08:04

viewer   ~~0023149

This patch only remove Stream BASS erros in Console and move to:

onClientSoundStream(bool success, int length, string streamName, string error)

http://pastebin.com/ARCYKRE5

arranTuna

2015-04-20 12:16

manager   ~~0023217

https://github.com/multitheftauto/mtasa-blue/commit/971dc486f85acbc33cf970c88924ca97dbefd922

Issue History

Date Modified Username Field Change