Skip to content

[DC-49] feature: start removing support for OC10 #12167

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

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Conversation

modSpike
Copy link
Contributor

…DetermineAuthTypeAdapter

the AuthenticationType enum lives in the adapter for now, but this is temporary.
also added a few todo's from the refactoring todo list
Copy link

update-docs bot commented May 26, 2025

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@modSpike modSpike self-assigned this May 26, 2025
@modSpike modSpike marked this pull request as draft May 26, 2025 09:50
modSpike added 11 commits May 26, 2025 19:00
there are a few traces left in the basic http/credentials code but that is going away in the next commit
…f legacy webfinger support)

also improved the docs for the new adapters
… obvious guis

also did some various cleanup, renaming, and streamlining
still need to tighten up the handling of the authentication type in HttpCredentials as this is the most logical "owner" of that value.
… to credentialssupport.h

removed the DetermineAuthTypeJob class
fixed various includes while removing overuse of #include httpcredentials.h
use it to check whether the server supports spaces in the final auth checks. If not, the server is rejected and no account is created.
added a few todo's
cleaned up dead comments.
still need to test this on kw (I don't anticipate any issues there) and also with webfinger
@modSpike modSpike marked this pull request as ready for review June 4, 2025 16:54
@modSpike modSpike requested a review from DeepDiver1975 June 5, 2025 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants