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

PreviousPageURI may not work #2

Open
kevinburke opened this issue Oct 25, 2016 · 2 comments
Open

PreviousPageURI may not work #2

kevinburke opened this issue Oct 25, 2016 · 2 comments

Comments

@kevinburke
Copy link
Collaborator

If you go to Page=2 by following next_page_uri links, then try to get the previous_page_uri, you get back a 400 from the Twilio API. I've written the API team to ask why that is.

@kevinburke
Copy link
Collaborator Author

API team is aware of this issue and working on a fix

kevinburke added a commit that referenced this issue Nov 11, 2016
It's broken in the Twilio API so there's not much we can do about it until they
fix it. Matching on the string because we don't have an error code to match on.
@kevinburke
Copy link
Collaborator Author

Commit 5724a23 now links to this issue if you hit this error in Logrole. Unfortunately the error doesn't have an associated error code, so I'm matching based on the error string:

    str := strings.Replace(err.Error(), "twilio: ", "", 1)
    if strings.Contains(strings.ToLower(str), "aftersid is required for paging") {
        str = str + " See https://github.com/saintpete/logrole/issues/2"
    }

(cc @dougblack)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant