Comments (2)
Thanks for creating this issue! I will add a few comments to clarify our situation.
Facebook makes it sound like app secret proof is optional in their docs, however, as we can see from the email they sent us, app secret proof is mandatory for any app that wants to go into production mode:
![image](https://private-user-images.githubusercontent.com/7578559/338669084-e168d686-9158-463c-aa87-fcbaaed12106.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjE0NTgyNzAsIm5iZiI6MTcyMTQ1Nzk3MCwicGF0aCI6Ii83NTc4NTU5LzMzODY2OTA4NC1lMTY4ZDY4Ni05MTU4LTQ2M2MtYWE4Ny1mY2JhYWVkMTIxMDYucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDcyMCUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA3MjBUMDY0NjEwWiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9YjA4MTFiZDVlOGY3MGU3M2U1MmVlNzE3NTYwNjgyODgwNGYyMGZlNGU0YzU5MDA1ZTFkMDg1ZWJlOGRmMTE4MyZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.cZ6SONTic0qyfpLQeOoktpqaG5FgIkjQt0HGiyEPCiU)
So, in order to make uppy.io work,
AND, in general, to make any app that uses Uppy + Facebook in production work,
we will need to implement app secret proof support in Uppy.
from uppy.
Fixed by #5249
from uppy.
Related Issues (20)
- Some thoughts about the UI of GooglePhotos.
- Done button shows when doneButtonHandler is null
- SyntaxError in Safari on macOS when loading uppy.min.js (v3.22.2 and later) HOT 3
- Please add documentation on how to use uppy with an existing s3 client HOT 1
- Tus endpoint not being respected after initial request behind reverse proxy HOT 9
- i18n "Shared with me" etc HOT 1
- Allow streaming upload also for unknown length streams HOT 1
- @uppy/companion: include origin in dynamic customer credentials
- Using Vue2 to integrate uppy caused a strange mistake HOT 2
- aws-nodejs-example | Multipart complete api | Throws 400 bad requests. HOT 5
- title.value is not accessible inside uppy.upload in vue HOT 1
- I get this error "ResizeObserver loop completed with undelivered notifications" when I set width and height 100% in Angular HOT 5
- Uncaught TypeError: Cannot read properties of null (reading 'postMessage') in Companion's Callback Endpoint HOT 10
- Fix TypeScript sins HOT 5
- v4: typescript error for `uppy.setFileState()`
- Typescript error for 'new Uppy().use(AwsS3, {})' HOT 2
- XHR plugin expects upload response to be a valid JSON HOT 8
- companionAllowedHosts is not implemented in RemoteSources preset
- Uppy companion server error HOT 4
- Uppy 4.0.5 error HOT 5
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from uppy.