Skip to content

Uploading new files post migration to own s3 bucket not saving on parse.com #1808

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

Closed
kvreem opened this issue May 16, 2016 · 9 comments
Closed

Comments

@kvreem
Copy link

kvreem commented May 16, 2016

  • Server: parse server, heroku for deployment
  • Database: mLab shared plan
  • iOS application Parse 1.13 written in swift

Problem Occurs When:

  • User who is on new infrastructure changes their profile picture
  • A new user signs up on new infrastructure and uploads there own profile picture

Problem:
On my new infrastructure all new files are being stored in my s3 bucket. All users who haven't updated will not be able to see these changes. I looked at the parse.com dashboard to see what was happening and this was what I found.

screen shot 2016-05-16 at 11 12 01 am

The file that says "ilePicture.png" is the newly changed file. So it is updating that column in my database. However when I go to download it I get this:

screen shot 2016-05-16 at 11 12 09 am

For users on the new infrastructure they can get the new and old files. But for those who have not updated they cannot see the new s3 files.

@drew-gross
Copy link
Contributor

Probably related to parse-community/parse-dashboard#351

@kvreem
Copy link
Author

kvreem commented May 16, 2016

@drew-gross can you elaborate what they mean by use indexOf? My app is an iOS app on Parse SDK 1.13. Would the way I save the files changes in my client code? Or is this a change in parse-server itself?

@hramos
Copy link
Contributor

hramos commented May 16, 2016

This appears to be a duplicate of issue #1582

@kvreem
Copy link
Author

kvreem commented May 16, 2016

@hramos any idea on the time frame or if a fix can be posted here on the comments? @flovilmart said he was looking at it 25 days ago

from parse-community/parse-dashboard#351

"Possible solution is to avoid the length based calculation, and use indexOf(""), and this will be a good solution for it, but needs to check first if tfss is found, since there, theres no "" separating the name from the uuid."

@flovilmart
Copy link
Contributor

@kkhattab did't get a chance to have a look

@kvreem
Copy link
Author

kvreem commented May 16, 2016

@flovilmart no problem, i am currently blocked from pushing the migrated version until then, my team and i found that reverting back to the previous version and trying to change the image immediately crashes the app. i can follow up again here, any estimated time frame?

@hramos
Copy link
Contributor

hramos commented May 17, 2016

I'll go ahead and close this, feel free to continue the discussion in #1582. I don't think the issue is being actively worked on at this time, but we'd be happy to take a look at any pull request you may send our way to address it.

@hramos hramos closed this as completed May 17, 2016
@kvreem
Copy link
Author

kvreem commented May 17, 2016

@hramos forgive me if i am mistaken but looking at the discussions and related issues this seems to be a big deal. Could a time range be estimated for this issue? It can be very insightful for other people who come across this issue to know. Thanks!

@kvreem
Copy link
Author

kvreem commented Jun 1, 2016

@hramos just wanted to check up on this issue, any status updates? any alternative solutions in the mean time? thanks!

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

4 participants