Feature/remove rest client from celery #726
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Kudos, SonarCloud Quality Gate passed!
|
Pull Request Test Coverage Report for Build 4590
💛 - Coveralls |
Sandro-Meireles
approved these changes
Jul 21, 2022
helllllllder
added a commit
that referenced
this pull request
Jul 21, 2022
* Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * Develop (#709) * Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module (#712) * Feature/internal endpoint organization (#713) * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * Feature/internal endpoint repository (#714) * Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * add repository endpoints * code smells * duplicated lines * fix code smells * Feature/internal endpoint user (#715) * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * add repository endpoints * add user endpoints * flake8 and black * Feature/connect rest client (#716) * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * add repository endpoints * add user endpoints * flake8 and black * implement connect rest client * fix code smells * black * Feature/add csp (#717) * Develop (#709) (#710) * Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * add regex remotion of special characters from username when creating from keycloak * add django_csp and settings * configure csp * fix settings * black * Feature/internal endpoint user (#718) * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * add repository endpoints * add user endpoints * flake8 and black * fix user endpoints * Feature/add csp (#719) * Develop (#709) (#710) * Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * add regex remotion of special characters from username when creating from keycloak * add django_csp and settings * configure csp * fix settings * black * fix csp settings * fix csp settings * Feature/internal endpoint user (#720) * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * add repository endpoints * add user endpoints * flake8 and black * fix user endpoints * fix connect client * Fix the repository get_user_authorization method The updated method compares repository and organization authorizations and uses the highest role level without overwriting the repository's authorization level * Update rule with repository authorization role overwriting The method now overwrites the repository's authorization role if is lower than 4 (i.e. ROLE_TRANSLATOR), as of this level on, it no longer has corresponding roles at the client app. * Linting * feat: update list_project_organization endpoint removing the celery call * feat: update list_project_organization endpoint removing the celery call (#725) * feat: update all code that called celery tasks for communicating with connect to calls directly into ConnectRESTClient * Feature/remove rest client from celery (#726) * feat: update list_project_organization endpoint removing the celery call * feat: update all code that called celery tasks for communicating with connect to calls directly into ConnectRESTClient * bugfix: remove task.result after condition to user or not grpc endpoints, as it was conflicting with the rest call * feat: cache request result in list_classifiers Co-authored-by: Victor Salles <victorsalles1997@gmail.com> Co-authored-by: vitusalis <49169676+vitusalis@users.noreply.github.com>
helllllllder
added a commit
that referenced
this pull request
Jul 27, 2022
* Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module (#712) * Feature/internal endpoint organization (#713) * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * Feature/internal endpoint repository (#714) * Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * add repository endpoints * code smells * duplicated lines * fix code smells * Feature/internal endpoint user (#715) * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * add repository endpoints * add user endpoints * flake8 and black * Feature/connect rest client (#716) * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * add repository endpoints * add user endpoints * flake8 and black * implement connect rest client * fix code smells * black * Feature/add csp (#717) * Develop (#709) (#710) * Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * add regex remotion of special characters from username when creating from keycloak * add django_csp and settings * configure csp * fix settings * black * Feature/internal endpoint user (#718) * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * add repository endpoints * add user endpoints * flake8 and black * fix user endpoints * Feature/add csp (#719) * Develop (#709) (#710) * Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * add regex remotion of special characters from username when creating from keycloak * add django_csp and settings * configure csp * fix settings * black * fix csp settings * fix csp settings * Feature/internal endpoint user (#720) * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * add repository endpoints * add user endpoints * flake8 and black * fix user endpoints * fix connect client * Fix the repository get_user_authorization method The updated method compares repository and organization authorizations and uses the highest role level without overwriting the repository's authorization level * Update rule with repository authorization role overwriting The method now overwrites the repository's authorization role if is lower than 4 (i.e. ROLE_TRANSLATOR), as of this level on, it no longer has corresponding roles at the client app. * Linting * feat: update list_project_organization endpoint removing the celery call (#725) * Feature/remove rest client from celery (#726) * feat: update list_project_organization endpoint removing the celery call * feat: update all code that called celery tasks for communicating with connect to calls directly into ConnectRESTClient * Feature/remove rest client from celery (#727) * feat: update list_project_organization endpoint removing the celery call * feat: update all code that called celery tasks for communicating with connect to calls directly into ConnectRESTClient * bugfix: remove task.result after condition to user or not grpc endpoints, as it was conflicting with the rest call * Feature/remove rest client from celery (#730) * Develop (#709) (#710) * Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * feat: update list_project_organization endpoint removing the celery call * feat: update all code that called celery tasks for communicating with connect to calls directly into ConnectRESTClient * bugfix: remove task.result after condition to user or not grpc endpoints, as it was conflicting with the rest call * feat: cache request result in list_classifiers * Feature/remove rest client from celery (#731) * Develop (#709) (#710) * Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * feat: update list_project_organization endpoint removing the celery call * feat: update all code that called celery tasks for communicating with connect to calls directly into ConnectRESTClient * bugfix: remove task.result after condition to user or not grpc endpoints, as it was conflicting with the rest call * feat: cache request result in list_classifiers * feat: delete cache when creating and removing a classifier * Revert "Feature/remove rest client from celery (#731)" (#733) This reverts commit 4a0b7f2. * Revert "Feature/remove rest client from celery (#730)" (#734) This reverts commit 8e42c7d. Co-authored-by: Victor Salles <victorsalles1997@gmail.com> Co-authored-by: vitusalis <49169676+vitusalis@users.noreply.github.com>
helllllllder
added a commit
that referenced
this pull request
Jul 27, 2022
* Develop (#709) * Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * feat: Change gRPC call to REST calls * Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module (#712) * Feature/internal endpoint organization (#713) * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * Feature/internal endpoint repository (#714) * Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * add repository endpoints * code smells * duplicated lines * fix code smells * Feature/internal endpoint user (#715) * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * add repository endpoints * add user endpoints * flake8 and black * Feature/connect rest client (#716) * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * add repository endpoints * add user endpoints * flake8 and black * implement connect rest client * fix code smells * black * Feature/add csp (#717) * Develop (#709) (#710) * Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * add regex remotion of special characters from username when creating from keycloak * add django_csp and settings * configure csp * fix settings * black * Feature/internal endpoint user (#718) * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * add repository endpoints * add user endpoints * flake8 and black * fix user endpoints * Feature/add csp (#719) * Develop (#709) (#710) * Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * add regex remotion of special characters from username when creating from keycloak * add django_csp and settings * configure csp * fix settings * black * fix csp settings * fix csp settings * Feature/internal endpoint user (#720) * Create internal module, module permission and updated WeniOIDCAuthenticationBackend to add permission if the user is a module * Add organization internal endpoints * add repository endpoints * add user endpoints * flake8 and black * fix user endpoints * fix connect client * Fix the repository get_user_authorization method The updated method compares repository and organization authorizations and uses the highest role level without overwriting the repository's authorization level * Update rule with repository authorization role overwriting The method now overwrites the repository's authorization role if is lower than 4 (i.e. ROLE_TRANSLATOR), as of this level on, it no longer has corresponding roles at the client app. * Linting * feat: update list_project_organization endpoint removing the celery call (#725) * Feature/remove rest client from celery (#726) * feat: update list_project_organization endpoint removing the celery call * feat: update all code that called celery tasks for communicating with connect to calls directly into ConnectRESTClient * Feature/remove rest client from celery (#727) * feat: update list_project_organization endpoint removing the celery call * feat: update all code that called celery tasks for communicating with connect to calls directly into ConnectRESTClient * bugfix: remove task.result after condition to user or not grpc endpoints, as it was conflicting with the rest call * Feature/remove rest client from celery (#730) * Develop (#709) (#710) * Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * feat: update list_project_organization endpoint removing the celery call * feat: update all code that called celery tasks for communicating with connect to calls directly into ConnectRESTClient * bugfix: remove task.result after condition to user or not grpc endpoints, as it was conflicting with the rest call * feat: cache request result in list_classifiers * Feature/remove rest client from celery (#731) * Develop (#709) (#710) * Fix: Remove ai from project (#707) * remove document deletion from delete_nlp_logs task * inconsistency number and debug errors fixed * add user_email to remove_authorizations_project * Feature/health check blocklist (#708) * remove document deletion from delete_nlp_logs task * add a blocklist for not saving logs depending on the authorization user * inconsistency number and debug errors fixed * change the REPOSITORY_BLOCK_USER_LOGS values from users to repository authorizations * change readme * pass on sonarcloud * change admins settings * transform uuid into string * convert uuid into string at test_blocked_user * feat: update list_project_organization endpoint removing the celery call * feat: update all code that called celery tasks for communicating with connect to calls directly into ConnectRESTClient * bugfix: remove task.result after condition to user or not grpc endpoints, as it was conflicting with the rest call * feat: cache request result in list_classifiers * feat: delete cache when creating and removing a classifier * Revert "Feature/remove rest client from celery (#731)" (#733) This reverts commit 4a0b7f2. * Revert "Feature/remove rest client from celery (#730)" (#734) This reverts commit 8e42c7d. Co-authored-by: Victor Salles <victorsalles1997@gmail.com> Co-authored-by: vitusalis <49169676+vitusalis@users.noreply.github.com> Co-authored-by: Victor Salles <victorsalles1997@gmail.com> Co-authored-by: vitusalis <49169676+vitusalis@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WHAT
Update every call to celery for communicating with connect to call directly ConnecRESTClient if the env var USE_GRPC is False
WHY
Solve a cache problem with celery, it was saving the old keycloak token