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

Can't see photos: 301 Moved Permanently CloudFront #82

Open
eduardogoncalves opened this issue Feb 15, 2020 · 3 comments · May be fixed by #85
Open

Can't see photos: 301 Moved Permanently CloudFront #82

eduardogoncalves opened this issue Feb 15, 2020 · 3 comments · May be fixed by #85

Comments

@eduardogoncalves
Copy link

Wittr is not displaying images:
Captura de Tela 2020-02-15 às 15 51 29

When I try to access image's url it show a 301 message.

Captura de Tela 2020-02-15 às 15 45 15

/ping @jakearchibald

@caleb-train
Copy link

I'm having the same issue, please is there any solution to this yet.

@bontaramsonta
Copy link

please update the urls to point to new images desitnations

@firedev
Copy link

firedev commented Nov 29, 2020

Try this

/server/Server.js:101

    this._app.get('/photos/:farm-:server-:id-:secret-:type.jpg', (req, res) => {
       const flickrUrl = `http://farm${req.params.farm}.staticflickr.com/${req.params.server}/${req.params.id}_${req.params.secret}_${imgSizeToFlickrSuffix[req.params.type]}.jpg`;
       const flickrRequest = http.request(flickrUrl, flickrRes => {
-        flickrRes.pipe(res);
+        Object.keys(flickrRes.headers).forEach((header) => {
+          res.setHeader(header, flickrRes.headers[header]);
+        });
+        // flickrRes.pipe(res);
+        res.sendStatus(301)
       });

firedev added a commit to firedev/wittr that referenced this issue Dec 1, 2020
@firedev firedev linked a pull request Dec 1, 2020 that will close this issue
gwpantazes pushed a commit to gwpantazes/wittr-modern that referenced this issue Mar 7, 2023
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

Successfully merging a pull request may close this issue.

4 participants