Skip to content
This repository has been archived by the owner on Jun 1, 2024. It is now read-only.

chore: merge 1.4.0 into master #1082

Merged
merged 18 commits into from
Oct 26, 2020
Merged

chore: merge 1.4.0 into master #1082

merged 18 commits into from
Oct 26, 2020

Conversation

uziasferreirazup
Copy link
Contributor

@uziasferreirazup uziasferreirazup commented Oct 26, 2020

Explanation:

Last week, we had to make a release to solve some potential security problems for users that don't customize beagle core mechanisms (network client, logger, cache) with their own implementation. Since it's impractical to have a default implementation that would satisfy completely different security concerns, we decided to remove our default implementation of these mechanisms, and let users implement their own with security concerns bounded to their specific context.

Since our master branch had features being developed, we were unable to generate a release from it. Therefore a new branch was created from the last release (1.3.0), which also incorporates hot fixes from both platforms.

Now with this PR, this code with security changes will be integrated into master.

matheusribeirozup and others added 17 commits October 16, 2020 09:41
* fix: parse color when is RGB size

Signed-off-by: Matheus Ribeiro <matheus.ribeiro@zup.com.br>

* refactor: change method structure in ColorUtils.kt

Signed-off-by: Matheus Ribeiro <matheus.ribeiro@zup.com.br>

* refactor tests method names

Signed-off-by: Matheus Ribeiro <matheus.ribeiro@zup.com.br>
* refactor: remove deprecated from constructor

* fix: adjust name action backend

* fix: adjust serialization context

* adjust try serializer

* adjust context data manager

* adjust try to deserializer

* adjust unit tests

* adjust class

* remove space extra

* adjust test in form

* adjust evaluation

* adjust lint

* fix: adjust context data evaluation to enum

* adjust lint
* crash app in cache

* private val in ContextBinding

Co-authored-by: Frederico Noce <p000rbertholucci@prservicos.com.br>
* fix: BeagleFragment breaking change

Signed-off-by: paulomeurerzup <paulo.meurer@zup.com.br>

* adjust test methods name

Signed-off-by: paulomeurerzup <paulo.meurer@zup.com.br>

* remove unnused imports

Signed-off-by: paulomeurerzup <paulo.meurer@zup.com.br>
…ger (#1070)

* fix: remove default implementation

* adjust tests

* add error when api it is empty

* adjust unit tests

* adjust unit tests

* adjust message

* adjust unit test

* adjust unit test

* format url

* adjust comment
…ger (#1073)

* remove network client from dependencies

* remove DefaultCache and DefaultLogger from Beagle

* add defult implementations on demo

* fix tests compiling issues and remove CacheDiskManager from Beagle

* fix BeagleDependencies tests

* handle network client nil

* remove main storyboard from BeagleDemo

* reorganizing BeagleDemo file structure

* SendRequest now executes error when there is some infrastructure problem

* fix code smell

* fix code smell

Co-authored-by: Daniel Tes Carrasque <daniel@zup.com.br>
Co-authored-by: Lucas Araújo <lucas.araujo@zup.com.br>
@github-actions github-actions bot added android This issue directly affects structure and logic of Android project iOS This issue directly affects structure and logic of iOS project labels Oct 26, 2020
@botbeaglezup
Copy link
Contributor

1 Warning
⚠️ Big PR

Generated by 🚫 Danger

@sonarcloud
Copy link

sonarcloud bot commented Oct 26, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 36 Code Smells

No Coverage information No Coverage information
1.9% 1.9% Duplication

@botbeaglezup
Copy link
Contributor

1 Warning
⚠️ android/beagle/src/main/java/br/com/zup/beagle/android/data/serializer/PolymorphicJsonAdapterFactory.java#L308 - Implicitly using the default locale is a common source of bugs: Use toLowerCase(Locale) instead. For strings meant to be internal use Locale.ROOT, otherwise Locale.getDefault().

Generated by 🚫 Danger

@theffc theffc changed the title fix: merge 1.4.0 into master chore: merge 1.4.0 into master Oct 26, 2020
@uziasferreirazup uziasferreirazup merged commit 4e34608 into master Oct 26, 2020
@uziasferreirazup uziasferreirazup deleted the fix/merge-1.4.0 branch October 26, 2020 15:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
android This issue directly affects structure and logic of Android project iOS This issue directly affects structure and logic of iOS project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants