Skip to content
This repository was archived by the owner on Dec 3, 2020. It is now read-only.

Authentication replace #7

Merged
merged 28 commits into from
Oct 11, 2019
Merged

Authentication replace #7

merged 28 commits into from
Oct 11, 2019

Conversation

parssinen
Copy link

Summary

Modifies node server to not redirect to Helsinki authentication server but instead use basic auth.

Dependencies

Testing instructions

Checklist for pull request creator

  • A task has been created for the PR on the Kanban board with necessary details filled (one task / repo)
  • The commits and commit messages adhere to version control conventions
  • The code is consistent with the existing code base
  • Tests have been written for the change
  • All tests pass (unit, e2e)
  • All code has been linted and there aren't any lint errors
  • The change has been tested in the browser with Firefox 59+, Chrome 66+
  • The change has been tested with a smaller screen (tablet)
  • The change conforms to the UX specifications
  • All translations have been added (fi, sv, en)
  • The code is self-documenting or has been documented sufficiently, e.g., in the README
  • The branch has been rebased against master before the PR was created

Checklist for pull request reviewer (copy to review text box)

- [ ] A task has been created for the PR on the Kanban board with necessary details filled (one task / repo)
- [ ] The commits and commit messages adhere to [version control conventions](https://voltti.atlassian.net/wiki/spaces/NUORA/pages/32999/Versionhallintak+yt+nn+t#Versionhallintak%C3%A4yt%C3%A4nn%C3%B6t-Commit-viestienmuotoilu)
- [ ] The code is consistent with the existing code base
- [ ] All changes in all changed files have been reviewed
- [ ] Tests have been written for the change
- [ ] All tests pass (unit, e2e)
- [ ] All code has been linted and there aren't any lint errors
- [ ] The change has been tested in the browser with Firefox 59+, Chrome 66+
- [ ] The change has been tested with a smaller screen (tablet)
- [ ] The change conforms to the [UX specifications](https://voltti.atlassian.net/wiki/spaces/NUORA/pages/74809369/UX)
- [ ] All translations have been added (fi, sv, en)
- [ ] The code is self-documenting or has been documented sufficiently, e.g., in the README
- [ ] The PR branch has been rebased against master and force pushed if necessary before merging

Lauri Nuotio and others added 28 commits October 11, 2019 14:44
Upon clicking Login-button, prompt username and password and send them using basic auth against modified auth server
Upon clicking Login-button, prompt username and password and send them using basic auth against modified auth server
Copy link

@kuosmark kuosmark left a comment

Choose a reason for hiding this comment

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

  • A task has been created for the PR on the Kanban board with necessary details filled (one task / repo)
  • The commits and commit messages adhere to version control conventions
  • The code is consistent with the existing code base
  • All changes in all changed files have been reviewed
  • Tests have been written for the change
  • All tests pass (unit, e2e)
  • All code has been linted and there aren't any lint errors
  • The change has been tested in the browser with Firefox 59+, Chrome 66+
  • The change has been tested with a smaller screen (tablet)
  • The change conforms to the UX specifications
  • All translations have been added (fi, sv, en)
  • The code is self-documenting or has been documented sufficiently, e.g., in the README
  • The PR branch has been rebased against master and force pushed if necessary before merging

@kuosmark kuosmark merged commit 929796f into master Oct 11, 2019
@kuosmark kuosmark deleted the auth-replace branch October 11, 2019 13:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants