[D9] OpenID Connect HarID

in your composer.json please also add the drupal core dependency

{
  "name": "drupal/openid_connect_harid",
  "type": "drupal-module",
  "description": "OpenID Connect HarID Client",
  "keywords": ["Drupal", "OpenID Connect", "HarID", "Client"],
  "license": "MIT",
  "homepage": "https://github.com/sisuloome/openid_connect_harid",
  "minimum-stability": "dev",
  "support": {
    "issues": "https://github.com/sisuloome/openid_connect_harid/issues",
    "source": "https://github.com/sisuloome/openid_connect_harid"
  },
  "require": {
    "drupal/core": "^8.8 || ^9",
    "drupal/openid_connect" : "^1.1"
  }

Automated test

Passed

Manual test

Individual user account:Yes
No duplication:Yes
Master Branch:Yes
Licensing:Yes
3rd party assets/code:Yes
README.txt/README.md:Yes
Code long/complex enough for review:Yes
Secure code: Yes
Coding style & Drupal API usage:
- Recommendation: in the .module

  function openid_connect_harid_openid_connect_post_authorize(UserInterface $account, array $context) {
  if ($context['plugin_id'] === 'harid') {
    if (isset($context['userinfo']['ui_locales'])) {
      // TODO See if there should be a check for allowed values of en, et, ru.
      $account->set('langcode', $context['userinfo']['ui_locales']);
      $account->set('preferred_langcode', $context['userinfo']['ui_locales']);
      $account->save();
    }
  }

can go under one if statement.

- Recommendation in the .module and .install
Replace \Drupal::
By adding on top
use Drupal
It is used more that 1 time so this implementation will have a faster execution time.

openid_connect_harid.info.yml
Consider placing the min php version to 7.2 as it is the min recommended version of Drupal 8.8

OpenIDConnectHarIDClient.php
In this part of the code

public function getEndpoints() {
    $base_url = $this->getBaseUrl();

    return [
      'authorization' => $base_url . '/authorizations/new',
      'token' => $base_url . '/access_tokens',
      'userinfo' => $base_url . '/user_info',
    ];
  }

use Url::fromUri
https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Url.php/f...

As for the Url::fromUrl
This is from the documentation:

Same name and namespace in other branches
Creates a new Url object from a URI.

This method is for generating URLs for URIs that:
do not have Drupal routes: both external URLs and unrouted local URIs like base:robots.txt

Replacing \Drupal with use Drupal.
Each time you use a \Drupal you are trying to access the Class or call it.
When you have it one time in the code it makes sense that it is only in that method/function.
When it is repeating it is better to "use Drupal"
Basically it is like you are setting an variable and just reusing it vs. reinitializing the variable.
Hope that this makes sense.

That was more of an advice than an error. It was returned because of the Url Class.

I have rechecked both automated test and redid a manual checkup. I see no other problems here.Unless someone else finds someting.

Thank you for your contribution! I am going to update your account.

These are some recommended readings to help with excellent maintainership:

• Dries' post on Responsible maintainers
• Best practices for creating and maintaining projects
• Maintaining a drupal.org project with Git
• Commit messages - providing history and credit
• Release naming conventions.
• Helping maintainers in the issue queues

You can find more contributors chatting on the IRC #drupal-contribute channel. So, come hang out and stay involved.
Thank you, also, for your patience with the review process.
Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

I thank all the dedicated reviewers as well.