toxblh/podcasts
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

DataError: Improve the FeedRedirect variant

Browse Source
This commit is contained in:
Jordan Petridis 2018-09-04 12:41:37 +03:00
parent a61bbc460f
commit ca6996e53a
No known key found for this signature in database
GPG Key ID: E8523968931763BE
2 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
podcasts-data/src/errors.rs
View File

@ -63,8 +63,8 @@ pub enum DataError {
Bail(String),
#[fail(display = "{}", _0)]
HttpStatusGeneral(HttpStatusError),
#[fail(display = "FIXME: This should be better")]
F301(Source),
#[fail(display = "Source redirects to a new url")]
FeedRedirect(Source),
#[fail(
display = "Error occured while Parsing an Episode. Reason: {}",
reason

11
podcasts-data/src/models/source.rs
View File

@ -146,13 +146,14 @@ impl Source {
match code {
StatusCode::NotModified => return Err(self.make_err("304: skipping..", code)),
StatusCode::MovedPermanently | StatusCode::Found | StatusCode::PermanentRedirect => {
error!("Feed was moved permanently.");
warn!("Feed was moved permanently.");
self = self.update_url(&res)?;
return Err(DataError::F301(self));
return Err(DataError::FeedRedirect(self));
}
StatusCode::TemporaryRedirect => {
debug!("307: Temporary Redirect.");
return Err(DataError::F301(self));
warn!("307: Temporary Redirect.");
// FIXME: How is it actually handling the redirect?
return Err(DataError::FeedRedirect(self));
}
StatusCode::Unauthorized => return Err(self.make_err("401: Unauthorized.", code)),
StatusCode::Forbidden => return Err(self.make_err("403: Forbidden.", code)),
@ -212,7 +213,7 @@ impl Source {
.then(|res| match res {
Ok(response) => Ok(Loop::Break(response)),
Err(err) => match err {
DataError::F301(s) => {
DataError::FeedRedirect(s) => {
info!("Following redirect...");
Ok(Loop::Continue(s))
}