-
Notifications
You must be signed in to change notification settings - Fork 105
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
Error: Match error: Expected plain object #2
Comments
This is a problem with the check function and Safari |
just filed issue with Meteor project meteor/meteor#3207 |
This needs a workaround. Thanks for reporting. |
Should be fixed now. |
Thank you |
Since MDG just closed the issue with the check function I looked into this again and got it to work right with the following test on all browsers (Firefox, Safari, Chrome)
|
Previously, HTTPS certificate validation would fail if dots were in the bucket name because the bucketURL would then contain multiple levels of subdomains (e.g. some.depth.of.domains.s3.amazonaws.com). Now, the bucket name is not included in the domain name during uploads, and the HTTPS certificate is valid even when dots are in the bucket name. Note that this commit does not fix the situation for downloads. This is more complicated because of multiple ways of downloading a file. With S3 alone there is CulturalMe#1 ( https://bucket-name.s3[-and-possibly-region].amazonaws.com/key ) and the approach used by this commit for uploading CulturalMe#2 ( https://s3[-and-possibly-region].amazonaws.com/bucket-name/key ). Then there is with a CDN, which would typically be something like CulturalMe#3 ( https://cdn-distribution-name.cloudfront.net/key ). A suitable solution to create download URLs that allows dots in the bucket name is not covered by this commit. To word around this present limitation, construct your own download URL using options CulturalMe#1, CulturalMe#2 and CulturalMe#3 according to your needs.
Try this in Safari: |
When I get to this part in my example
it always fails with error in view.js:406
The text was updated successfully, but these errors were encountered: