Update source and order of profile data fetching in preparation for D9 accounts
In preparation for the D9 migration, a few things need to be updated with regards to the order and source of the aggregated profile data.
A list of items to handle:
- Point to D9 accounts on staging when available
-
Add
full_name
to fields retrieved from LDAP, as well as final return object -
Fetch
publisher_agreements
,org
andorg_id
data from fdndb-api-
If user doesn't have
EMPLY
role with an Eclipse org, useorg
andorg_id
from accounts
-
If user doesn't have
-
Change order of operations to:
- Fetch from LDAP first, 404 if not found
- Fetch from accounts, 404 if not found
- Fetch remaining info from fdndb-api, if available
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Zachary Sabourin added prioritymedium severitylow statewip team:webdev labels
added prioritymedium severitylow statewip team:webdev labels
- Zachary Sabourin assigned to @zacharysabourin
assigned to @zacharysabourin
- Zachary Sabourin changed the description
Compare with previous version changed the description
- Zachary Sabourin mentioned in commit zacharysabourin/eclipsefdn-profile-api@f34f2d89
mentioned in commit zacharysabourin/eclipsefdn-profile-api@f34f2d89
- Zachary Sabourin mentioned in merge request !35 (merged)
mentioned in merge request !35 (merged)
- Zachary Sabourin marked the checklist item Add
full_name
to fields retrieved from LDAP, as well as final return object as completedmarked the checklist item Add
full_name
to fields retrieved from LDAP, as well as final return object as completed - Zachary Sabourin marked the checklist item Fetch from LDAP first, 404 if not found as completed
marked the checklist item Fetch from LDAP first, 404 if not found as completed
- Zachary Sabourin marked the checklist item Fetch from accounts, 404 if not found as completed
marked the checklist item Fetch from accounts, 404 if not found as completed
- Zachary Sabourin mentioned in commit zacharysabourin/eclipsefdn-api-common@dbe8b2a7
mentioned in commit zacharysabourin/eclipsefdn-api-common@dbe8b2a7
- Zachary Sabourin mentioned in merge request eclipsefdn-api-common!140 (merged)
mentioned in merge request eclipsefdn-api-common!140 (merged)
- Zachary Sabourin mentioned in commit zacharysabourin/eclipsefdn-api-common@5be63dcd
mentioned in commit zacharysabourin/eclipsefdn-api-common@5be63dcd
- Zachary Sabourin changed the description
Compare with previous version changed the description
- Zachary Sabourin added statereview label and removed statewip label
added statereview label and removed statewip label
- Maintainer
@zacharysabourin I just made available a light version of the user profile endpoint on accounts.eclipse.org. It's the same implementation from API.eclipse.org that I enabled on accounts.eclipse.org.
I am hoping that this is enough for you to enable you to go live with the profile API in Q4. I am assuming that you don't need to be able to search by email so oAuth is not enabled on accounts.eclipse.org.
https://accounts.eclipse.org/account/profile/cguindon
Note that the URL at the bottom don't work since I did not enable those endpoints. The API is simply picking up the base URL of the current site.
"forums_url": "https://accounts.eclipse.org/account/profile/cguindon/forum", "projects_url": "https://accounts.eclipse.org/account/profile/cguindon/projects", "gerrit_url": "https://accounts.eclipse.org/account/profile/cguindon/gerrit", "mailinglist_url": "https://accounts.eclipse.org/account/profile/cguindon/mailing-list", "mpc_favorites_url": "https://accounts.eclipse.org/marketplace/favorites/?name=cguindon"
I don't believe you need any of these for your service.
The D9 API will use the same URL /account/profile/:username but you should only expect the following data: https://gitlab.eclipse.org/eclipsefdn/it/websites/drupal/eclipsefdn/-/merge_requests/195/diffs#408482c0f693dcb7daa33e66275e197282e5c33a_0_108
We should aim at testing with D9 in Q1 2024.
Let me know if you have any question!
Collapse replies - Author Contributor
Perfect, sounds good!
I've already made the required changes to the process for handling the incoming data in my patch. I just need to actually point to accounts.
For github_id and email requests, I can fetch from LDAP and query accounts using the username. No problems there. Is there a way for me to fetch from accounts using the Drupal UID?
- Maintainer
No problems there. Is there a way for me to fetch from accounts using the Drupal UID?
I am on a call with Martin and he said the same thing :) I've enabled oAuth so you should be able to do that now.
- Author Contributor
Ok thanks!
- Please register or sign in to reply
- Zachary Sabourin marked the checklist item Fetch
publisher_agreements
,org
andorg_id
data from fdndb-api as completedmarked the checklist item Fetch
publisher_agreements
,org
andorg_id
data from fdndb-api as completed - Zachary Sabourin marked the checklist item If user doesn't have
EMPLY
role with an Eclipse org, useorg
andorg_id
from accounts as completedmarked the checklist item If user doesn't have
EMPLY
role with an Eclipse org, useorg
andorg_id
from accounts as completed - Zachary Sabourin marked the checklist item Change order of operations to: as completed
marked the checklist item Change order of operations to: as completed
- Zachary Sabourin marked the checklist item Fetch remaining info from fdndb-api, if available as completed
marked the checklist item Fetch remaining info from fdndb-api, if available as completed
- Zachary Sabourin marked this issue as blocking #70 (closed)
marked this issue as blocking #70 (closed)
- Zachary Sabourin added statewip label and removed statereview label
added statewip label and removed statereview label
- Zachary Sabourin marked the checklist item Point to D9 accounts on staging when available as completed
marked the checklist item Point to D9 accounts on staging when available as completed
- Zachary Sabourin added statereview label and removed statewip label
added statereview label and removed statewip label
- Zachary Sabourin mentioned in commit b6903a0c
mentioned in commit b6903a0c
- Zachary Sabourin closed with merge request !35 (merged)
closed with merge request !35 (merged)