friendica
/
friendica-addons
7
2
Fork 13
Code Issues 3 Pull Requests 2 Releases 16 Wiki Activity

Mobile client tokens addon #1115

Closed
very-ape wants to merge 0 commits from develop into develop
pull from: develop
merge into: friendica:develop
Conversation 7 Commits - Files Changed -
very-ape commented 2021-05-18 20:42:20 +02:00 (Migrated from github.com)
Copy Link

One of my members tried to use a mobile client yesterday and discovered my SSO scheme didn't play nicely with mobile clients like Fedilab, DiCa, and Friendiqa, as these clients require a username and password. Rather than create an addon to allow them to use their Keycloak credentials with Friendica (as that would compromise their accounts on every service I run in the event of, say, mobile malware), I wrote an addon to allow the issuance and revocation of secondary credentials for Friendica accounts.

One of my members tried to use a mobile client yesterday and discovered my SSO scheme didn't play nicely with mobile clients like Fedilab, DiCa, and Friendiqa, as these clients require a username and password. Rather than create an addon to allow them to use their Keycloak credentials with Friendica (as that would compromise their accounts on every service I run in the event of, say, mobile malware), I wrote an addon to allow the issuance and revocation of secondary credentials for Friendica accounts.
very-ape commented 2021-05-18 20:43:21 +02:00 (Migrated from github.com)
Copy Link

Alright then. I have no idea why that second CI check is failing. I'll wait for input.

Alright then. I have no idea why that second CI check is failing. I'll wait for input.
MrPetovan commented 2021-05-18 20:44:17 +02:00 (Migrated from github.com)
Copy Link

We are using application passwords for the built-in two-factor authentication feature, is there a way you can leverage the existing feature?

We are using application passwords for the built-in two-factor authentication feature, is there a way you can leverage the existing feature?
very-ape commented 2021-05-18 20:49:04 +02:00 (Migrated from github.com)
Copy Link

I'll familiarize myself with that feature and see if there's some way I can rewrite this addon to use that infrastructure instead. There doesn't seem to be a way to achieve what I'm looking for without some sort of addon, though.

I'll familiarize myself with that feature and see if there's some way I can rewrite this addon to use that infrastructure instead. There doesn't seem to be a way to achieve what I'm looking for without some sort of addon, though.
MrPetovan commented 2021-05-18 20:54:57 +02:00 (Migrated from github.com)
Copy Link

You are right, at the very least the app specific password table can be used instead of the general purpose config table.

You are right, at the very least the app specific password table can be used instead of the general purpose config table.
👍 1
tobiasd commented 2021-05-18 20:58:02 +02:00 (Migrated from github.com)
Copy Link

Alright then. I have no idea why that second CI check is failing. I'll wait for input.

Ignore it ;-) that test is being worked on and currently it is not working.

> Alright then. I have no idea why that second CI check is failing. I'll wait for input. Ignore it ;-) that test is being worked on and currently it is not working.
👍 1
very-ape commented 2021-05-18 21:29:35 +02:00 (Migrated from github.com)
Copy Link

Wow, yeah. This is extremely close to what I'm looking for. Mostly the trouble seems UI-related. I'm going to go in a slightly different direction now. Will hopefully have a pair of addons ready for review in a day or two.

Edit: Also, I notice now that I forgot to make sure "token IDs" didn't allow slashes in them, which would've caused problems.

Wow, yeah. This is extremely close to what I'm looking for. Mostly the trouble seems UI-related. I'm going to go in a slightly different direction now. Will _hopefully_ have a pair of addons ready for review in a day or two. Edit: Also, I notice now that I forgot to make sure "token IDs" didn't allow slashes in them, which would've caused problems.
👍 1
MrPetovan commented 2021-05-18 21:34:04 +02:00 (Migrated from github.com)
Copy Link

Thank you!

Thank you!
👍 1

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
2018.09

   
2019.01

   
2019.03

   
2019.06

   
2019.09

   
2019.12

   
2020.03

   
2020.06

   
2020.09

   
2020.12

   
2021.03

   
2021.07

   
2021.09

   
2022.02

   
2022.06

   
2022.09

   
2022.12

   
2023.04

   
2023.05

   
2023.09

   
2024.03

   
2024.06

   
dependencies

Pull requests that update a dependency file

  
Hackathon 2021
No Label
2018.09
2019.01
2019.03
2019.06
2019.09
2019.12
2020.03
2020.06
2020.09
2020.12
2021.03
2021.07
2021.09
2022.02
2022.06
2022.09
2022.12
2023.04
2023.05
2023.09
2024.03
2024.06
dependencies
Hackathon 2021
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: friendica/friendica-addons#1115
No description provided.
Delete Branch "develop"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?