simplesamlphp / simplesamlphp/simplesamlphp-module-negotiate
Checking Windows machines domain membership
Nessuno ha ancora preso questa issue.
- Lingua principale
- PHP
- Stelle
- 4
- Fork
- 5
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
I noticed in the documentation mention of the issue with Windows machines popping up NTLM authentication boxes, which rather negates the point of the Negotiate module. We hit this some years ago with earlier version of simpleSAMLphp, and we couldn't just limit it by subnet as our users use a mix of organisationally owned domain bound machines and their own devices when on site.
I wrote a little work round that uses an extra, optional string in the negotiate module configuration in authsources.php called checkDomain. This is the AD domain name that the machine should be in. I then added an extra routine to the src/Auth/Source/Negotiate.php code to check this, with a call just after the subnet mask checks. It uses /usr/bin/nmblookup to search for the Windows machine, and ignores browsers that don't include "Windows NT" in their user agent string (so Linux boxes, Macs, etc).
In case it is of use, here's a diff of the changes against a recent version of the negotiate module:
$ diff Negotiate.php.dist Negotiate.php
58d57
<
82a82
> $this->check_domain = $cfg->getOptionalString('checkDomain', NULL);
131a132,139
> // Check if client is in allowed AD domain
> $domainOK = $this->checkDomain();
> if (!$domainOK) {
> Logger::debug('Negotiate - Not a matching domain. falling back');
> $this->fallBack($state);
> return;
> }
>
226a235,267
> /**
> * checkDomain() looks up the domain that the client is bound into
> * and limits clients to the specified domain.
> *
> * Will return TRUE if no domain restriction option is configured.
> *
> * @return boolean
> */
> public function checkDomain(): bool {
> // No domain means all clients are accepted.
> if ($this->check_domain === NULL)
> return true;
>
> // Only do this check for Windoze machines.
> $ua = $_SERVER['HTTP_USER_AGENT'];
> if (!preg_match("/Windows NT/", $ua)) {
> return true;
> }
>
> $ip = $_SERVER['REMOTE_ADDR'];
> $cmd = "/usr/bin/timeout 1 /usr/bin/nmblookup -A $ip 2>&1";
> $shellOutput = shell_exec($cmd);
> error_log($shellOutput,0);
> $lines = explode(PHP_EOL, $shellOutput);
> $myDomain = $this->check_domain;
> foreach ($lines as $line) {
> if(preg_match("/$myDomain\s+\<00\>\s\-\s\<GROUP\>.+ACTIVE/",
> $line)) {
> return true;
> }
> }
> return false;
> }
We've been using this in production on SSP 1.x IdP servers for many years now, and I've just ported it ready for use to move to SSP 2.x.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Leggi src/Auth/Source/Negotiate.php e la configurazione authsources.php, iniziando dai controlli di sottorete esistenti e dal diff fornito. Verifica come l'impostazione opzionale checkDomain dovrebbe interagire con gli user agent Windows NT, nmblookup e il comportamento di fallback; il lavoro è completato quando la configurazione documentata e il controllo del dominio sono integrati con una verifica appropriata.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- php
- Ambito
- authentication
- Tipo di issue
- Funzionalità
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 35/100