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

Featured images missing in Reader #21709

Open
nbradbury opened this issue Feb 25, 2025 · 5 comments · May be fixed by #21713
Open

Featured images missing in Reader #21709

nbradbury opened this issue Feb 25, 2025 · 5 comments · May be fixed by #21713
Assignees
Labels

Comments

@nbradbury
Copy link
Contributor

As noted here:

I noted one issue occurring on trunk that appears unrelated to these changes: None of the WordPress.com News post featured images loaded in the Android app. Featured images from other sites—e.g., The Atavist Magazine—do load.

@dangermattic
Copy link
Collaborator

Thanks for reporting! 👍

@nbradbury
Copy link
Contributor Author

Example: iOS Reader

Image

Android Reader:

Image

@nbradbury
Copy link
Contributor Author

It appears that when loading featured images here, the image isn't being shown. Examples below work fine in the browser but not the app:

https://en-blog.files.wordpress.com/2025/02/what-is-domain-privacy.jpg?strip=info&quality=65&w=1280

https://en-blog.files.wordpress.com/2025/02/blog-header-v2.jpg?strip=info&quality=65&w=1280

@nbradbury
Copy link
Contributor Author

nbradbury commented Feb 26, 2025

Problem occurs not just in reader detail but also reader list and reader photo viewer. Logcat shows this:

NetworkDispatcher.processRequest: Unhandled exception java.lang.IllegalStateException: cannot make a new request because the previous response is still open: please call response.close()

This happens with posts from WordPress.com News but not other blogs I tried.

@nbradbury
Copy link
Contributor Author

nbradbury commented Feb 26, 2025

Possibly (likely?) related to the above, it appears our CustomRedirectInterceptor doesn't close the existing connection upon redirect.

Update: Closing the existing connection does solve this problem, but not sure of any side effects.

@nbradbury nbradbury linked a pull request Feb 26, 2025 that will close this issue
@nbradbury nbradbury self-assigned this Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants