Improved handling of VoD file metadata, added some failsaves against metadata corruption.

This commit is contained in:
Thulinma 2014-11-11 15:42:17 +01:00
parent 455bce2c34
commit 318af0b353
2 changed files with 47 additions and 21 deletions

View file

@ -344,6 +344,9 @@ int Controller::handleAPIConnection(Socket::Connection & conn){
Controller::fillTotals(Request["totals"], Response["totals"]);
}
}
Controller::writeConfig();
}else{//unauthorized
Util::sleep(1000);//sleep a second to prevent bruteforcing
logins++;

View file

@ -108,6 +108,11 @@ namespace Controller {
}
data["online"] = 0;
return;
}else{
if (!getMeta && data["meta"].packedSize() > 1*1024 * data["meta"]["tracks"].size()){
DEBUG_MSG(DLVL_WARN, "Metadata for stream %s is quite big (%u b) - assuming corruption and forcing reload", name.c_str(), data["meta"].packedSize());
getMeta = true;
}
}
}else{
DEBUG_MSG(DLVL_INSANE, "Invalid metadata (no tracks object) for stream %s - triggering reload", name.c_str());
@ -135,9 +140,28 @@ namespace Controller {
data["online"] = 0;
return;
}
unsigned int i = 0;
JSON::fromDTMI((const unsigned char*)streamIndex.mapped + 8, streamIndex.len - 8, i, data["meta"]);
if (data["meta"].isMember("tracks") && data["meta"]["tracks"].size()){
for(JSON::ObjIter trackIt = data["meta"]["tracks"].ObjBegin(); trackIt != data["meta"]["tracks"].ObjEnd(); trackIt++){
trackIt->second.removeMember("fragments");
trackIt->second.removeMember("keys");
trackIt->second.removeMember("parts");
trackIt->second.removeMember("ivecs");/*LTS*/
}
}
if ( !data["meta"] || !data["meta"].isMember("tracks") || !data["meta"]["tracks"]){
data["error"] = "Stream offline: Corrupt file?";
if (data["error"].asStringRef() != prevState){
Log("WARN", "Source file " + URL + " seems to be corrupt.");
}
data["online"] = 0;
return;
}
DEBUG_MSG(DLVL_INSANE, "Metadata for stream %s (re)loaded", name.c_str());
}
DEBUG_MSG(DLVL_INSANE, "Stream %s opened", name.c_str());
}
}else{
//now, run mistinfo on the source - or on the accompanying dtsh file, if it exists
if (stat((URL+".dtsh").c_str(), &fileinfo) == 0){
DEBUG_MSG(DLVL_INSANE, "Stream %s has a DTSH - opening DTSH instead of main stream file", name.c_str());
@ -159,6 +183,7 @@ namespace Controller {
}
DEBUG_MSG(DLVL_INSANE, "Metadata for stream %s succesfully (re)loaded", name.c_str());
}
}
if (!hasViewers(name)){
if ( !data.isMember("error")){
data["error"] = "Available";
@ -256,7 +281,6 @@ namespace Controller {
out[jit->first] = jit->second;
out[jit->first]["name"] = jit->first;
Log("STRM", std::string("Updated stream ") + jit->first);
checkStream(jit->first, out[jit->first]);
}
}else{
out[jit->first]["name"] = jit->first;
@ -268,7 +292,6 @@ namespace Controller {
out[jit->first]["cut"] = jit->second["cut"].asInt();
}
Log("STRM", std::string("New stream ") + jit->first);
checkStream(jit->first, out[jit->first]);
}
}
}