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

ACL feature #42

Closed
mcanini opened this issue Mar 12, 2015 · 4 comments
Closed

ACL feature #42

mcanini opened this issue Mar 12, 2015 · 4 comments

Comments

@mcanini
Copy link

mcanini commented Mar 12, 2015

I'd like to be able to specify which users are allowed to view and submit tasks for my course.
Ideally, I would like to do it by dropping a CSV file into my course git repository. The format of the CSV file is the one I download from the faculty.

This is the file format:
(Skip the first line, e.g. "LINGI2347 en 2014")
Pgrm,Num fac,Nom,Prénom,Noma,Mail,...

The # of columns can vary but the above ones are fixed.

I think from Noma and email address you can recognize a valid UCL / INGI username.

Thanks for implementing this!

@layus
Copy link
Contributor

layus commented Mar 12, 2015

+:100:! This would be a most useful feature.

If I understood well, this is difficult because authentication is based on logins, which is the only column not provided by the csv ?

@mcanini
Copy link
Author

mcanini commented Mar 12, 2015

I don't think it is difficult. But you need to have a way to figure out the correct mapping using an authoritative source.

@GuillaumeDerval
Copy link
Member

The verification by NOMA will not be possible directly:

  • INGInious uses LDAP to check user credentials; however, the NOMA does not seem to be stored on the INGI LDAP server, and the UCL one does not give the information too.
  • NOMA is a "UCL-only" feature; we try to keep INGInious agnostic and filtering by NOMA would break this.

However, it seems possible to filter:

  • by login
  • by email
    Easily :-)

@GuillaumeDerval
Copy link
Member

This will be available soon on our instance of INGInious in INGI.

(:tada: closed issue 42! :tada:)

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

No branches or pull requests

3 participants