Conversation
Member
smokhov
commented
Mar 14, 2026
smokhov
commented
- Not to merge; for conflict and clean up management
Implemented initial setup for the XML parsing. Approved-by: Miqdad Amirali <m_amira@encs.concordia.ca> Approved-by: Ryan Hotton <r_hotton@encs.concordia.ca>
updated the set up to include the interface and sub classes Approved-by: Ryan Hotton <r_hotton@encs.concordia.ca> Approved-by: Sophia Kavousifard <s_kavou@encs.concordia.ca>
…term. Tested it using the RSS parser.
… clashes with Ryan's XPath implementation.
Added XPath parsing and made a class for it so it can be called anywhere. Approved-by: Alex Genio <a_genio@encs.concordia.ca> Approved-by: George Grafos <g_grafos@encs.concordia.ca>
Implemented the SAX parser class to be used with or without a search term. Tested it using the RSS parser. Approved-by: Ryan Hotton <r_hotton@encs.concordia.ca> Approved-by: George Grafos <g_grafos@encs.concordia.ca>
…structions on how to set up each part.
…ormat and invokes a live-test in streamalert.
…ing the pathToStreamAlert variable and fixed a refactoring error.
…for sending parsed xml to slack via streamalert service.
update read me with soph's changes Approved-by: Alex Genio <a_genio@encs.concordia.ca> Approved-by: Ryan Hotton <r_hotton@encs.concordia.ca>
…738222 (pull request #15) README.md edited online with Bitbucket
…162080 (pull request #16) README.md edited online with Bitbucket
PM1 Part 3 -> PM1 branch Approved-by: Miqdad Amirali <m_amira@encs.concordia.ca>
…t using a search term when testing Part 3.
# Conflicts: # .gitignore # README.md
Team 02 - Pull Request - PM1
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.