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

Alpha - Support python lang #237

Merged
merged 58 commits into from
Feb 10, 2023
Merged

Alpha - Support python lang #237

merged 58 commits into from
Feb 10, 2023

Conversation

tuxology
Copy link
Contributor

No description provided.

Copy link
Member

@khemrajrathore khemrajrathore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tuxology

  • Add new language python in PrivadoTag.scala --> Language.
  • Also make appropriate change in ScanProcessor.scala --> parseRule --> langToFilter

@tuxology
Copy link
Contributor Author

Currently, because we don't have standard APIs for Python frontend, we get only base cpg with no overlays on it. We can handle running other overlays of course in Privado, but this is non-standard and will cause issue if upstream API changes eventually. Therefore, I'll be working on this as a dependency for continuing work here: joernio/joern#2113

@tuxology tuxology changed the title [WIP] Experimental python support Experimental python support Dec 26, 2022
@tuxology tuxology force-pushed the python-experimental branch from 57f355b to 0f148d5 Compare January 20, 2023 20:35
build.sbt Outdated Show resolved Hide resolved
dbMundada and others added 25 commits February 6, 2023 13:38
Add PythonAPITagger + update pyton processor logs
Remove 'json' from the python API tagger
Add support to match httpclient + BeautifulSoup
Fix API sink pattern to support f'' strings in python3
@dbMundada dbMundada changed the title Experimental python support Alpha - Support python lang Feb 10, 2023
@dbMundada dbMundada dismissed khemrajrathore’s stale review February 10, 2023 08:28

Confirmed with Khemraj looks good.

@pandurangpatil pandurangpatil merged commit a86dd97 into dev Feb 10, 2023
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.

7 participants