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

Release 2020/07/13 #1164

Merged
merged 68 commits into from
Jul 14, 2020
Merged

Release 2020/07/13 #1164

merged 68 commits into from
Jul 14, 2020

Conversation

fisx
Copy link
Contributor

@fisx fisx commented Jul 13, 2020

Release Notes

  • If you are self-hosting wire on the public internet, consider changing your brig server config.
  • Deploy all services except nginz.
  • No migrations, no restrictions on deployment order.

New Features

Bug Fixes

Internal changes

Jon Elverkilde and others added 30 commits April 26, 2018 11:11
* Remove redundant cabal file from version control.

* Set stack resolver to match https://github.com/wireapp/wire-server/.

* Eliminate servant dependency exception.

(this will be helpful if we link this against the same executable
as saml2-web-sso and don't want to run two servants at the same time.)

* Cleanup

* Enable travis ci.

* Add missing test cases to Spec.hs.

* Use hspec-discover
* Use UUIDs

* Add a comment

* Add missing fields to 'ListResponse'

* Serve a single user

* Give the user an email

* Resource types

* Address comments
* A module for filtering users

* Integrate filtering into the mock implementation

* Don't allow users with duplicate usernames

* Overhaul error handling; fix comments
)

* Implement basic auth in a simpler way (without contexts etc)

* Throw errors via ExceptT

* Cleanup CCP switch for servant compatibility. (#13)

* Throw authorization errors as 'SCIMError's

* Move language extensions to package.yaml

* Export siteServer

* Add 'serverError'

* Make 'decodeAuth' more informative

* Remove TestStorage duplication

* Add all the necessary instances for FromJSON StoredUser
* Add 'renderFilter'

* Add a roundtrip test
Needed for LTS-12 (servant-generic is deprecated).
* Clarify 'version'

* Don't merge fields in PUT /Users

* Fix a warning

* Redo User serialization

* More tests for new User serialization
* Support extra fields in 'User'

* Comments
* More comments; don't send the lowercase version to FromJSON extra

* Clarify
* Add 'forbidden' error helper
* Customizable ID types + no extra logic

* Fix the Id type (it must behave like a string)

* Don't test how we handle duplicate usernames

* Migrate groups

* Inline mock server methods
* ListResponse is 1-indexed, not 0-indexed

* implement patch in terms of get and put

* Implements patch _only_ for userName, displayName, externalId

* Add Azure acceptance test

Co-authored-by: fisx <mf@zerobuzz.net>
* Make hscim compile with LTS 14.12

We need this to bump wire-server

* Add version bounds

This makes it possible to also build this with Cabal and get a similar
build plan :)

* Don't ignore cabal file

I think we even need it if we need to upload this to Hackage?

Sure hpack is cute, but for applications... Also I'm not sure
if I buy the yaml all the things trend

* Update gitignore

* Don't hardcode LTS version in CI? what's the point of that :)
Co-Authored-By: fisx <mf@zerobuzz.net>
mheinzel and others added 19 commits June 24, 2020 11:16
* test invalid email or phone

* move validateDomain
* Add test

* Make 'active' field patchable

* Make tests marginally more generic.

Co-authored-by: Matthias Fischmann <mf@zerobuzz.net>
* fix galley services-demo config

* align nginx-docker.conf with nginx.conf

* align other services' Docker config with non-Docker version
* separate FeatureStatus into sum type and wrapper again

* Move json string into query param (or keep it as an object).

Co-authored-by: Matthias Fischmann <mf@zerobuzz.net>
git-subtree-dir: libs/hscim
git-subtree-mainline: 8710776
git-subtree-split: f5f7489
They led to errors (and displaying undefined) in swagger-ui.

Now there are still 3 cases left where it shows undefined instead of the model,
but it seems to be a problem with using returning an array of references
like `Doc.returns (Doc.array (Doc.ref Public.modelFoo))`.
* add a Back Office tab to swagger-ui

* make stern endpoints reachable from UI

* add --run-backoffice flag to services-demo

* make nginx start even if stern is not there

* explain how to run the Back Office locally
- global stack.yaml, stack.yaml.lock.
- remove stack noise from package.
- remove stack*, .travis.yml, ormolu.sh from package.
- Remove doctests.
The missing ok from CLAAssistant is due to the fact that Jon Elverkilde, the
author of many commits in hscim, has changes his email address and thus
the commits don't match the github user any more.  Jon was an employee of
wire at the time of writing this code and has signed of his IP in the employment
contract, so this should not be an issue.
* galley: add migration

* galley: add DB queries

* galley: implement creating mapping

* galley: implement resolving opaque IDs

* galley: add internal endpoints for debugging/testing

* galley: do intra call (where the Brig endpoint is still missing)

* galley: add minimal integration tests

* galley: rename fields of IdMapping

* galley: throw federation-not-enabled

* update cassandra-schema.cql
* Add active flag to SCIM user

* Implement tests and calls to brig
  Involved exposing some types in brig that were internal before.

* Remove misguided TODO.
  https://github.com/wireapp/wire-server/blob/b9a84f9b654a69c9a296761b36c042dc993236d3/services/brig/src/Brig/Team/API.hs#L306

* Promote NOTE to TODO; make it clear which line it belongs to.

* Scim onboarding: handle all AccountStatus values reasonably.

* Clarifying comment.

* Fix test case docs.

* Better mapping: scim active flag <-> AccountStatus.
  both more correct (does not fail on valid requests) and
  more readable, maintainable.

* Cleanup.

* Fix account status mapping for bogus ephemeral case.

Co-authored-by: Matthias Fischmann <mf@zerobuzz.net>
* squash #483

* remove all usages of aws

* support s3DownloadEndpoint config

* stack.yaml: pin updated fork of amazonka

The update to stack.yaml just takes all amazonka packages from the pinned fork.
They depend on core, so need to be rebuilt anyways.
We also drop elb, redshift, route53

* depend on OpenSSL again

This is mostly done for consistency with the other services. If we want to change it,
we should change it everywhere.

* remove unnecessary dependencies on binary and blaze-builder

* Add comment on how to use cloudfront

Co-authored-by: Tiago Loureiro <tiago.loureiro@gmail.com>
* docs.

* option.

* application logic.

* tests.
@CLAassistant
Copy link

CLAassistant commented Jul 13, 2020

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
7 out of 8 committers have signed the CLA.

✅ fisx
✅ jschaul
✅ ChrisPenner
✅ arianvp
✅ akshaymankar
✅ flokli
✅ mheinzel
❌ Jon Elverkilde


Jon Elverkilde seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@fisx
Copy link
Contributor Author

fisx commented Jul 13, 2020

I took out #1157 since it's a bit fresh and not urgent. Instead of leaving it out of the branch, which would have caused (minor) merge conflicts, I reverted it on top of the release branch. Once the release is merged back into develop, we need to add another un-revert commit to take it back in.

@fisx
Copy link
Contributor Author

fisx commented Jul 13, 2020

CLAAssistant is confused about John's identity due to data missing on github, and can be ignored.

Copy link
Contributor

@mheinzel mheinzel left a comment

Choose a reason for hiding this comment

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

Also, please squash all the CHANGELOG commits into one at the end.

CHANGELOG.md Outdated Show resolved Hide resolved
@fisx fisx force-pushed the release_2020_07_13 branch from 925736d to a43a215 Compare July 14, 2020 07:44
@fisx fisx requested a review from mheinzel July 14, 2020 07:44
@fisx fisx merged commit 0269dfa into master Jul 14, 2020
@fisx fisx deleted the release_2020_07_13 branch July 14, 2020 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants