Resolve "Improve security by enabling and enforcing more linting rules"

Description

See #242 (closed)

added componentci goSDN prioritylow statusavailable typeimprovement labels

assigned to @istfaseid

added 1 commit

• 0c9c3aa4 - enable errcheck for linter

Compare with previous version

added 1 commit

• 93539735 - removed type assertion checking from errcheck, started fixing missing error handling

Compare with previous version

added 3 commits

• d3a7883e - 1 commit from branch develop
• cc48f476 - Merge branch 'develop' into 242-improve-security-by-enabling-and-enforcing-more-linting-rules
• e47817df - more error handling, everything except event publishing

Compare with previous version

added 2 commits

• 57d017b5 - handle missing errors, could need some improvement
• 27f2b62b - some code simplification, enabled type assertion in errcheck again

Compare with previous version

added 1 commit

• a324c537 - more linters and candidates to consider, fixed issue with multiplication of time.Duration

Compare with previous version

added 1 commit

• 96380d79 - enabled error linting and fixed problems with wron error checks

Compare with previous version

added 1 commit

• 6858f7da - more linters

Compare with previous version

added 1 commit

• fd7c7a12 - more linters, also spellchecker with autofix

Compare with previous version

added 1 commit

• 5e5e0c6c - added missing event publishing to routing table

Compare with previous version

added 1 commit

• bd347753 - added and removed some linters, fiexed occuring issues

Compare with previous version

added 1 commit

• ee754470 - fix failing test

Compare with previous version

added 2 commits

• 5f445a11 - 1 commit from branch develop
• 8ba2a426 - Merge branch 'develop' into 242-improve-security-by-enabling-and-enforcing-more-linting-rules

Compare with previous version

marked this merge request as ready

requested review from @istaester and @ext10578

Please also check the few added //nolint during the review. Maybe there would be a better way to actually handle the issues instead of ignoring them

added 1 commit

• a795f618 - removed todo statement in error handling, create issue instead

Compare with previous version

added 2 commits

• 93f48c81 - 1 commit from branch develop
• 8738ae37 - Merge branch 'develop' into 242-improve-security-by-enabling-and-enforcing-more-linting-rules

Compare with previous version

mentioned in issue #249 (closed)

resolved all threads

resolved all threads

added 1 commit

• ee3ea48b - just added another optional linter

Compare with previous version

added 1 commit

• 34bfda04 - improved error handling of defer functions to include possible previous errors

Compare with previous version

added 2 commits

• 8ed37da0 - 1 commit from branch develop
• 379af765 - Merge branch 'develop' into 242-improve-security-by-enabling-and-enforcing-more-linting-rules

Compare with previous version

added 1 commit

• 02352b3b - linter said no

Compare with previous version

approved this merge request

LGTM

added 1 commit

• 2ffd42f0 - fugly godot commit

Compare with previous version

approved this merge request

merged

mentioned in commit 346cbe31