Skip to content

Fixing param type#735

Merged
Mzack9999 merged 1 commit intomainfrom
734-fix-memoize-param-type-extraction
Apr 22, 2026
Merged

Fixing param type#735
Mzack9999 merged 1 commit intomainfrom
734-fix-memoize-param-type-extraction

Conversation

@Mzack9999
Copy link
Copy Markdown
Member

Close #734

@neo-by-projectdiscovery-dev
Copy link
Copy Markdown

neo-by-projectdiscovery-dev Bot commented Mar 24, 2026

Neo - PR Security Review

No security issues found

Highlights

  • Fixes parameter type extraction in memoize code generator by replacing fmt.Sprint() with types.ExprString()
  • Adds test coverage to verify context.Context and other qualified types are correctly extracted
  • Changes affect build-time code generation, not runtime security boundaries

Comment @pdneo help for available commands. · Open in Neo

@Mzack9999 Mzack9999 merged commit 8edb92f into main Apr 22, 2026
8 checks passed
@Mzack9999 Mzack9999 deleted the 734-fix-memoize-param-type-extraction branch April 22, 2026 17:24
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.

Issue in param type extraction

2 participants