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

gdpr logs collection #2260

Merged
merged 3 commits into from
Apr 23, 2019
Merged

gdpr logs collection #2260

merged 3 commits into from
Apr 23, 2019

Conversation

DenysGonchar
Copy link
Collaborator

@DenysGonchar DenysGonchar commented Apr 12, 2019

currently get_logs_from_node/4 receives zipped logs from other nodes as rpc call return value. as agreed that is ok for the first version, if logs are too big this should be changed to some other delivery method (e.g. FTP)

for testing run:

tools/test-runner.sh --db --preset internal_mnesia --skip-small-tests --skip-cover -- gdpr:retrieve_personal_data:retrieve_logs

big_tests/tests/gdpr_SUITE.erl Outdated Show resolved Hide resolved
@@ -54,6 +54,9 @@
-export([config_state/0]).
-export([config_states/0]).

-export([all_cluster_nodes/0,
Copy link
Member

Choose a reason for hiding this comment

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

all_cluster_nodes/0 not used in the PR eventually

Copy link
Member

Choose a reason for hiding this comment

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

Still not resolved.

Copy link
Contributor

@aleklisi aleklisi left a comment

Choose a reason for hiding this comment

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

RP looks good 👍

big_tests/tests/gdpr_SUITE.erl Show resolved Hide resolved
src/ejabberd_loglevel.erl Outdated Show resolved Hide resolved
src/admin_extra/service_admin_extra_gdpr.erl Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Apr 16, 2019

Codecov Report

Merging #2260 into gdpr-retrieve will increase coverage by 48.09%.
The diff coverage is 88.88%.

Impacted file tree graph

@@                Coverage Diff                 @@
##           gdpr-retrieve    #2260       +/-   ##
==================================================
+ Coverage          30.73%   78.83%   +48.09%     
==================================================
  Files                334      333        -1     
  Lines              28833    28798       -35     
==================================================
+ Hits                8862    22703    +13841     
+ Misses             19971     6095    -13876
Impacted Files Coverage Δ
src/sasl/cyrsasl_external.erl 91.83% <ø> (+91.83%) ⬆️
src/jlib.erl 83.69% <100%> (+25.36%) ⬆️
src/sasl/cyrsasl.erl 88.23% <100%> (+8.82%) ⬆️
src/auth/ejabberd_auth_external.erl 32.07% <100%> (ø) ⬆️
src/auth/ejabberd_auth.erl 60.67% <100%> (+20.38%) ⬆️
src/auth/ejabberd_auth_ldap.erl 59.25% <100%> (+59.25%) ⬆️
src/auth/ejabberd_auth_pki.erl 16.66% <100%> (+16.66%) ⬆️
src/auth/ejabberd_auth_jwt.erl 89.13% <100%> (ø) ⬆️
src/mam/mod_mam_muc.erl 75% <100%> (+0.51%) ⬆️
src/auth/ejabberd_auth_http.erl 73.68% <100%> (ø) ⬆️
... and 267 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update da2a046...c06fa89. Read the comment docs.

@fenek fenek merged commit 53f911a into gdpr-retrieve Apr 23, 2019
@fenek fenek deleted the gdpr-retrive-logs branch April 23, 2019 15:55
@fenek fenek added this to the MongooseIM 3.3.0++ milestone Jun 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants