[PR #249] [CLOSED] Add LDAP Authentication #549

New issue

Closed

opened 2026-02-25 23:34:41 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 23:34:41 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/go-shiori/shiori/pull/249
Author: @ynsta
Created: 5/16/2020
Status: ❌ Closed

Base: master ← Head: ldap

📝 Commits (3)

367b0f8 ✨Add LDAP Authentication
a84303b ⬆Update dependencies and vendor dir
86e3729 🐋Improve Dockerfile and Add compose

📊 Changes

838 files changed (+504351 additions, -107 deletions)

View changed files

📝 Dockerfile (+13 -9)
➕ docker-compose.yml (+57 -0)
📝 go.mod (+24 -19)
📝 go.sum (+159 -46)
➕ internal/auth/auth.go (+119 -0)
➕ internal/auth/ldapauth.go (+161 -0)
📝 internal/cmd/root.go (+4 -4)
➕ internal/env/env.go (+66 -0)
➕ internal/env/example_env_test.go (+107 -0)
📝 internal/webserver/handler-api.go (+7 -29)
➕ vendor/github.com/PuerkitoBio/goquery/.gitattributes (+1 -0)
➕ vendor/github.com/PuerkitoBio/goquery/.gitignore (+16 -0)
➕ vendor/github.com/PuerkitoBio/goquery/.travis.yml (+17 -0)
➕ vendor/github.com/PuerkitoBio/goquery/LICENSE (+12 -0)
➕ vendor/github.com/PuerkitoBio/goquery/README.md (+183 -0)
➕ vendor/github.com/PuerkitoBio/goquery/array.go (+124 -0)
➕ vendor/github.com/PuerkitoBio/goquery/doc.go (+123 -0)
➕ vendor/github.com/PuerkitoBio/goquery/expand.go (+70 -0)
➕ vendor/github.com/PuerkitoBio/goquery/filter.go (+163 -0)
➕ vendor/github.com/PuerkitoBio/goquery/go.mod (+8 -0)

...and 80 more files

📄 Description

Hello,

Here a new PR for ldap authentication on your latest branch and without the config part.

All settings are given via environment vars.

I also added a docker-compose as an example with all env vars and a postgresql.

Regards,

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/go-shiori/shiori/pull/249 **Author:** [@ynsta](https://github.com/ynsta) **Created:** 5/16/2020 **Status:** ❌ Closed **Base:** `master` ← **Head:** `ldap` --- ### 📝 Commits (3) - [`367b0f8`](https://github.com/go-shiori/shiori/commit/367b0f8043b17423920a88f014f62ae2fd849d88) ✨Add LDAP Authentication - [`a84303b`](https://github.com/go-shiori/shiori/commit/a84303bbd6b9138c8dcef6f40b8e0cdc29cd3f0c) ⬆Update dependencies and vendor dir - [`86e3729`](https://github.com/go-shiori/shiori/commit/86e3729e7f60f2cb6c8ca65a027444847b166ee6) 🐋Improve Dockerfile and Add compose ### 📊 Changes **838 files changed** (+504351 additions, -107 deletions) <details> <summary>View changed files</summary> 📝 `Dockerfile` (+13 -9) ➕ `docker-compose.yml` (+57 -0) 📝 `go.mod` (+24 -19) 📝 `go.sum` (+159 -46) ➕ `internal/auth/auth.go` (+119 -0) ➕ `internal/auth/ldapauth.go` (+161 -0) 📝 `internal/cmd/root.go` (+4 -4) ➕ `internal/env/env.go` (+66 -0) ➕ `internal/env/example_env_test.go` (+107 -0) 📝 `internal/webserver/handler-api.go` (+7 -29) ➕ `vendor/github.com/PuerkitoBio/goquery/.gitattributes` (+1 -0) ➕ `vendor/github.com/PuerkitoBio/goquery/.gitignore` (+16 -0) ➕ `vendor/github.com/PuerkitoBio/goquery/.travis.yml` (+17 -0) ➕ `vendor/github.com/PuerkitoBio/goquery/LICENSE` (+12 -0) ➕ `vendor/github.com/PuerkitoBio/goquery/README.md` (+183 -0) ➕ `vendor/github.com/PuerkitoBio/goquery/array.go` (+124 -0) ➕ `vendor/github.com/PuerkitoBio/goquery/doc.go` (+123 -0) ➕ `vendor/github.com/PuerkitoBio/goquery/expand.go` (+70 -0) ➕ `vendor/github.com/PuerkitoBio/goquery/filter.go` (+163 -0) ➕ `vendor/github.com/PuerkitoBio/goquery/go.mod` (+8 -0) _...and 80 more files_ </details> ### 📄 Description Hello, Here a new PR for ldap authentication on your latest branch and without the config part. All settings are given via environment vars. I also added a docker-compose as an example with all env vars and a postgresql. Regards, --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>