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

Implement ``single_user = <email>`` config option. #2165

Merged
merged 2 commits into from Apr 14, 2016

Conversation

Projects
None yet
4 participants
@jmchilton
Copy link
Member

commented Apr 14, 2016

Let Planemo and other such applications serve a full-fledged, personalized Galaxy instance without having to worry about logins, etc....

jmchilton added some commits Apr 13, 2016

Implement config.single_user = <email> option.
Let Planemo and other such applications serve a full-fledged Galaxy instance without having to worry about logins, etc....

@erasche erasche self-assigned this Apr 14, 2016

@erasche

This comment has been minimized.

Copy link
Member

commented Apr 14, 2016

will hijack remote user mechanics

makes it sound like it will set use_remote_user = True, but that must be set manually. Shouldn't it completely hijack remote_user mechanics?

@jmchilton

This comment has been minimized.

Copy link
Member Author

commented Apr 14, 2016

@erasche Yeah - it makes a little more sense from a deployer perspective but would muddle the implementation a little. I'll make this change I guess.

jmchilton added a commit to jmchilton/galaxy that referenced this pull request Apr 14, 2016

@jmchilton jmchilton force-pushed the jmchilton:single_user branch from b7fc6ef to 2546030 Apr 14, 2016

@erasche

This comment has been minimized.

Copy link
Member

commented Apr 14, 2016

👍 on this. Waiting for tests to pass.

@erasche erasche merged commit 4ffee71 into galaxyproject:dev Apr 14, 2016

4 checks passed

api test Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
framework test Build finished.
Details
toolshed test Build finished.
Details
@jmchilton

This comment has been minimized.

Copy link
Member Author

commented Apr 14, 2016

Awesome - thanks for the great feedback and for the merge @erasche.

@jmchilton jmchilton deleted the jmchilton:single_user branch Apr 14, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.