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

Entrega do Projeto #5

Open
wants to merge 76 commits into
base: main
Choose a base branch
from
Open

Conversation

wellalencarweb
Copy link

Porque?

Transações estavam falhando

Como?

Ajustes, uso de SOLID, Object Calisthenics

Tarefa no Quadro

wellingtonalencar and others added 30 commits August 31, 2021 09:55
Create dockerfile
Create docker-compose.yml
crate config xdebug
Add coverage to make test
add secrets token acesss GPG
add secrets to ignore
Develop: Git Hub Action - Infection Minimun Coverage
Create Interface and Clients Checkers
Create Factory to Checkers
Implement FrauCheckerFactory
Create test to FraudChecker
create commmand to infection
Implement SOLID: ISP The Interface Segregation Principle
…Principle

Refactor after use SOLID: ISP The Interface Segregation Principle
…Principle

Refactor after use SOLID: ISP The Interface Segregation Principle
Adaptater to Interface Segregations
wellingtonalencar and others added 29 commits September 3, 2021 21:55
Create trait to Email
Apply use Trait Email
Initial solution to method create
Create Getters and Setters to Notification
Create index to test

Create index to test
Implement dispatcherNotify
Create Class  DispatcherNotification
Create DispatcherNotification
Increment more class to index
Create procedural execution
Implement new Class
Create Notifier Client
Create Tax manager Client
remove  folder and namespaces
Create a Executor for tests aplication
Remove bloc try/catch
fixes and implements new tests
Implement teste and new methods
@RicardoZandonaiPicPay
Copy link

Logo
Checkmarx One – Scan Summary & Details30a9c2fc-ee5f-4398-a90d-18da8b22b576

New Issues

Severity Issue Source File / Package Checkmarx Insight
HIGH Missing User Instruction /Dockerfile: 1 A user should be specified in the dockerfile, otherwise the image will run as root
MEDIUM Container Traffic Not Bound To Host Interface /docker-compose.yml: 7 Incoming container traffic should be bound to a specific host interface
MEDIUM Healthcheck Not Set /docker-compose.yml: 3 Check containers periodically to see if they are running properly.
MEDIUM Host Namespace is Shared /docker-compose.yml: 3 The hosts process namespace should not be shared by containers
MEDIUM Memory Not Limited /docker-compose.yml: 3 Memory limits should be defined for each container. This prevents potential resource exhaustion by ensuring that containers consume not more than t...
MEDIUM Networks Not Set /docker-compose.yml: 3 Setting networks in services ensures you are not using dockers default bridge (docker0), which shares traffic bewteen all containers.
MEDIUM Privileged Ports Mapped In Container /docker-compose.yml: 7 Privileged ports (1 to 1023) should not be mapped. Also you should drop net_bind_service linux capability from the container unless you absolutely ...
MEDIUM Security Opt Not Set /docker-compose.yml: 3 Attribute 'security_opt' should be defined.
MEDIUM Unpinned Actions Full Length Commit SHA /ci.yml: 18 Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...
LOW Container Capabilities Unrestricted /docker-compose.yml: 3 Some capabilities are not needed in certain (or any) containers. Make sure that you only add capabilities that your container needs. Drop unnecessa...
LOW Cpus Not Limited /docker-compose.yml: 3 CPU limits should be set because if the system has CPU time free, a container is guaranteed to be allocated as much CPU as it requests
LOW Healthcheck Instruction Missing /Dockerfile: 1 Ensure that HEALTHCHECK is being used. The HEALTHCHECK instruction tells Docker how to test a container to check that it is still working

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