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

Develop 3.0 292 solrupdate #767

Merged
merged 72 commits into from
Jun 16, 2020
Merged

Conversation

martinsumner
Copy link

This brings in the uplift of the solr version, and some changes to allow riak_test to run on OTP 20+.

However, there still are riak_test failures. The failures appear to fall into three broad categories:

  • handling of unicode has changed (in OTP perhaps) so all tests of non-ascii keys/buckets/values are failing;
  • changes in URL endpoints between solr versions
  • other deprecated features (e.g. tests which include Javascript map/reduce)

I think this should still be merged in, then we need to make the decision whether to:

  • exclude yokozuna from riak from develop-3.0 due to broken tests, but provide a branch with yokozuna which can be built from source
  • include yokozuna in riak and invest the effort to fix the broken tests

llelf and others added 20 commits March 7, 2019 13:44
It doesn't check user-provided path in any way.
This allows overriding any file on the system with riak permissions.
Security fixes

1. if HTTP API is exposed:

  - read any file on the system — via /search/extract, the error message leaks
    file content;

  - send HTTP «GET /» request to any host — by PUT/POSTing text/xml document,
    or via /search/extract.  This is also likely riak DoS if the host is
    attacker-controlled.

2. if PB API is exposed:

  - send HTTP «GET /» request to any host — by PUT/POST, see above.

3. if HTTP API is exposed:

  - override (with garbage) any file on the system with riak permissions.
Outstanding ssue with behaviours not being recognised
Also don't xref for locals_not_used as there are debug statements that are compiled to not warn in dialyzer - but will be flagged by xref
Changes required to support building of riak_test cases in OTP 20 - 22.
Align with other repos
Actually build solr again!
Where bench is expecting to find it
eqc tests will now run (and pass) using

`./rebar3 as eqc do eunit`
@martinsumner
Copy link
Author

There is also an issue with needing to include basho_bench as a dep in order to get riak_tests to run (many yokozuna riak_tests require basho_bench).

Copy link

@martincox martincox left a comment

Choose a reason for hiding this comment

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

As far as the Erlang stuff goes, which looks to be the usual OTP20+ / relx fixups, looks good to me. Can't really comment on the Java / Solr side of things, but happy for it to be merged.

@martinsumner martinsumner marked this pull request as ready for review June 16, 2020 09:25
@martinsumner martinsumner merged commit 8ac890c into develop-3.0 Jun 16, 2020
martinsumner added a commit to basho/riak that referenced this pull request Jul 9, 2020
Change commented yokozuna branch, as this basho/yokozuna#767 is now merged into develop-3.0, and new updates are being PR'd into 3.0 (e.g. basho/yokozuna#768)
martincox pushed a commit to bet365/riak that referenced this pull request Jul 9, 2020
Change commented yokozuna branch, as this basho/yokozuna#767 is now merged into develop-3.0, and new updates are being PR'd into 3.0 (e.g. basho/yokozuna#768)
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.

3 participants