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

Fix/cleanup Federated Identity/Authentication & move to awesome-sysadmin #729

Merged
merged 7 commits into from
Sep 5, 2016

Conversation

n8225
Copy link
Contributor

@n8225 n8225 commented Aug 27, 2016

Issues #35 #471

Removed OpenID/OpenID Connect as it appears to be a standard and not software.
Added Apachev1 to licenses for Openca or should this just fall under Apache?

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

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

  • Your submissions are formatted according to the guidelines.

    [Name](http://homepage/) - Short description. ([Demo](http://url.to/demo), [Source Code](http://url.of/source/code)) License Language``

  • Your additions are ordered alphabetically.

  • Your additions are free software, or if not they have been added to non-free.

  • Any licenses you have added are in our list of licenses.

  • You have searched the repository for any relevant issues or PRs.

  • Any category you are creating has the minimum requirement of 3 items.

Fixed licenses and languages.
Remove OpenID/OpenID Connect as it appears to be a standard and not
software.
Updated Source codes.
@Kickball
Copy link
Collaborator

All the changes look good, I'm not sure on the best thing for the Apache licenses. @nodiscc has been the specialist on licenses, it would be good to see what he thinks.

@nodiscc
Copy link
Member

nodiscc commented Aug 28, 2016

I think we should move OCSPD (licensed under Apache v1.0) to awesome-sysadmin (Identity management section), following the discussion in #704. I couldn't find a summary of the differences between Apache v1.0 and v2.0.

http://openca.dyndns.org:4443/projects/ocspd/

@nodiscc nodiscc self-assigned this Aug 28, 2016
@n8225
Copy link
Contributor Author

n8225 commented Aug 28, 2016

@n1trux @nodiscc Should the whole section be moved to https://github.com/n1trux/awesome-sysadmin#identity-management?

@nodiscc
Copy link
Member

nodiscc commented Aug 28, 2016

After giving it another look, I think the whole section should be moved to awesome-sysadmin. @n1trux please let us know if this is ok.

@n1trux
Copy link
Contributor

n1trux commented Aug 29, 2016

RADIUS / FreeIPA and similar systems are definitely Sysadmin topics, yes. Feel free to create a PR :)

Don't wonder when I'm not merging immediately, often I accumulate PRs and do them all at once when I have the time.

@Kickball
Copy link
Collaborator

Awesome-Sysadmin#174 moves this section over. I wasn't 100% sure on which categories the items should go in so it could probably do with a look over. @n1trux @nodiscc @n8225.

@n8225
Copy link
Contributor Author

n8225 commented Aug 30, 2016

Removed the entries that have been moved to Awesome-Sysadmin. I think we should wait to merge this one until its committed on Awesome-Sysadmin.

@Kickball
Copy link
Collaborator

Yeah it is best to wait until our PR on awesome sysadmin is merged.

@nodiscc nodiscc changed the title Fix Fix/cleanup Federated Identity/Authentication & move to awesome-sysadmin Aug 30, 2016
@nodiscc
Copy link
Member

nodiscc commented Sep 3, 2016

awesome-foss/awesome-sysadmin#174 merged

@Kickball
Copy link
Collaborator

Kickball commented Sep 5, 2016

Is there a reason we are keeping daloRADIUS in the section?

@n8225
Copy link
Contributor Author

n8225 commented Sep 5, 2016

Can't remember, I might have just overlooked it.

@Kickball
Copy link
Collaborator

Kickball commented Sep 5, 2016

Not a problem, I don't see any reason to keep it in there though.

@n8225
Copy link
Contributor Author

n8225 commented Sep 5, 2016

Removed

@Kickball
Copy link
Collaborator

Kickball commented Sep 5, 2016

Thanks, looks good to me now. 👍

@arylatt arylatt merged commit 8f74dbe into awesome-selfhosted:master Sep 5, 2016
@n8225 n8225 mentioned this pull request Sep 5, 2016
61 tasks
@n8225 n8225 deleted the Fix-ID branch October 24, 2016 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

5 participants