Skip to content

Conversation

mattheusv
Copy link

What was a problem?

Insider don't load files in repository if your name contains *git*, like username.github.io

How this PR fixes the problem?

This PR change the regex to ignore .git/ and allow **.git** folders, but I'think that we should improvement the mechanism of load files that need to be analyzed.

This PR also includes regex changes to include javascript files in the validation (other types of files were being included in the validation)

@htrgouvea htrgouvea changed the base branch from master to develop July 8, 2020 12:24
@htrgouvea
Copy link
Contributor

Hi @msAlcantara,

Whenever possible, do your PR on the develop branch please.

Thx

@mattheusv mattheusv closed this Jul 8, 2020
@mattheusv mattheusv reopened this Jul 8, 2020
@mattheusv
Copy link
Author

Ok, sorry about that, and I closed PR it unintentionally too.

@mattheusv mattheusv force-pushed the master branch 2 times, most recently from 5cbea45 to 27dfb40 Compare July 8, 2020 13:07
@mattheusv mattheusv merged commit 0f26cae into insidersec:develop Jul 8, 2020
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