769a6890e3
**Description** There is an issue with the Nextcloud Social app:  I've looked into it and found the cause to be most likely the Nginx configuration. You can see here the redirection doesn't take into account any GET parameters:  So it will always respond `{"message":"webfinger not supported"}`.  But when I try with a non-existent `.well-known`, I can see the redirection works as intended.  So my conclusion is that these redirections needs to be removed. **⚙️ Type of change** - [ ] ⚙️ Feature/App addition - [x] 🪛 Bugfix - [ ] ⚠️ Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] 🔃 Refactor of current code **🧪 How Has This Been Tested?** To test if the fix is effective you need to first install the Social app then test with your own user the url like this (replace `nextcloud-domain` and `AccountName` with correct values): https://nextcloud-domain/.well-known/webfinger?resource=acct:@AccountName@nextcloud-domain (the Social app should give you the Federated Cloud ID directly so you can use it there) Then it should give an output different than `{"message":"webfinger not supported"}`. And the ".well-known/webfinger isn't properly set up!" should not appear. **📃 Notes:** The last line `return 301 /index.php$request_uri;` should already cover any redirection and works properly. **✔️ Checklist:** - [x] ⚖️ My code follows the style guidelines of this project - [x] 👀 I have performed a self-review of my own code - [ ] #️⃣ I have commented my code, particularly in hard-to-understand areas - [ ] 📄 I have made corresponding changes to the documentation - [ ] ⚠️ My changes generate no new warnings - [x] 🧪 I have added tests to this description that prove my fix is effective or that my feature works - [x] ⬆️ I increased versions for any altered app according to semantic versioning **➕ App addition** If this PR is an app addition please make sure you have done the following. - [ ] 🪞 I have opened a PR on [truecharts/containers](https://github.com/truecharts/containers) adding the container to TrueCharts mirror repo. - [ ] 🖼️ I have added an icon in the Chart's root directory called `icon.png` |
||
|---|---|---|
| .. | ||
| NOTES.txt | ||
| _configmap.tpl | ||
| _cronjobs.tpl | ||
| _ingressInjector.tpl | ||
| _initPerms.tpl | ||
| _validation.tpl | ||
| _waitNextcloud.tpl | ||
| common.yaml | ||