Skip to content
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

SSO PWM #419

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

darkdragon-001
Copy link
Contributor

Thank you for taking the time to work on a PR for Awesome-Sysadmin!

To ensure your PR is dealt with swiftly please check the following:

  • Your additions are Free software
  • Software your are submitting is not your own, unless you have a healthy ecosystem with a few contributors (which aren't your sock puppet accounts).
  • Submit one item per pull request. This eases reviewing and speeds up inclusion.
  • Format your submission as follows, where Demo and Clients are optional.
    Do not add a duplicate Source code link if it is the same as the main link.
    Keep the short description under 80 characters and use sentence case
    for it, even if the project's webpage or readme uses another capitalisation.
    Demo links should only be used for interactive demos, i.e. not video demonstrations.
    - [Name](http://homepage/) - Short description, under 250 characters, sentence case. ([Demo](http://url.to/demo), [Source Code](http://url.of/source/code), [Clients](https://url.to/list/of/related/clients-or-apps)) `License` `Language`
  • Additions are inserted preserving alphabetical order.
  • Additions are not already listed at awesome-selfhosted
  • The Language tag is the main server-side requirement for the software. Don't include frameworks or specific dialects.
  • You have searched the repository for any relevant issues or PRs, including closed ones.
  • Any category you are creating has the minimum requirement of 3 items (see Add SSO services #409)
  • Any software project you are adding to the list is actively maintained.
  • The pull request title is informative, unlike "Update README.md".
    Suggested titles: "Add aaa to bbb" for adding software aaa to section bbb,
    "Remove aaa from bbb" for removing, "Fix license for aaa", etc.

Why it is awesome

This provider is so awesome because it allows you to maintain a single account but provide many independent services.

Copy link
Collaborator

@nodiscc nodiscc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @darkdragon-001 , since #459 we have new contribution guidelines in place, could you please take some time to answer the following questions as best you can? Thanks

  • Why is it awesome?
  • Have you used it? For how long?
  • Is this in a personal or professional setup?
  • How many devices/users/services/... do you manage with it?
  • Biggest pros/cons compared to other solutions?

@darkdragon-001
Copy link
Contributor Author

Why is it awesome?

  • extends established and stable applications (LDAP) with self-service to reduce support capacities
  • adds SSO interfaces to easily connect with other applications

Have you used it? For how long?
I tried it shortly but I then have been overvoted to go with a proprietary setup 😭

Is this in a personal or professional setup?
Organization of volunteers.

How many devices/users/services/... do you manage with it?
Several thousands.

Biggest pros/cons compared to other solutions?
Pro: Clean and fast building on top of existing and established software
Con: Less widespread than more established alternatives

Copy link
Collaborator

@nodiscc nodiscc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Waiting for additional reviewers

Copy link
Contributor

@Rabenherz112 Rabenherz112 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not entirely convinced that this should be included in the list. Firstly, it appears that you may not have extensive experience with it yourself. Additionally, when I review the issue tab, it seems that there are numerous unresolved issues, and the application receives only minor updates, mainly related to dependency updates.

@darkdragon-001
Copy link
Contributor Author

darkdragon-001 commented Sep 24, 2023

@Rabenherz112 I do see regular commits including bug fixes and huge refactorings in the commit history: https://github.com/pwm-project/pwm/commits/master.

To me it seems like the project already has a lot of great features. If people want additional features they are encouraged to send a PR with such addition and the maintainer seems very open to integrate proposed changes. Of course just opening an issue does not give any guarantee that someone else will work on it in their free time without any sort of reward.

@Rabenherz112
Copy link
Contributor

I share your perspective that the application appears to be quite mature and doesn't require as much maintenance as some other applications.

However, it does strike me as somewhat unusual that there are numerous open issues dating all the way back to 2016. That's why I am a bit unsure about it. I wholeheartedly agree that opening issues doesn't guarantee that they will be fixed or implemented, but there seems to be a limited response from the main maintainer regarding such issues. Additionally, certain issues appear to have been open for multiple years without any response, such as #57, #536, and several others (though I'm uncertain if this might be attributable to user errors).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants