Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds a new data source (InfoRos) to the suspicious sites manager, integrating domain names from OpenFacto's analysis of Russian-speaking websites associated with GRU activities.
Key Changes
- Added InfoRos as Source 9, registering domains from the "The GRU's Galaxy of Russian-speaking websites" analysis
- Configured source metadata including organization, report URL, and date
- Added console logging for the InfoRos source loading confirmation
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| // Source 9: InfoRos - noms de domaines | ||
| if (typeof infoRosDomains !== 'undefined' && Array.isArray(infoRosDomains)) { | ||
| this.registerSource('InfoRos Domains', infoRosDomains, { | ||
| name: 'Réseau InfoRos', |
There was a problem hiding this comment.
Extra space after the colon in 'name: ' property. Should be 'name: ' (single space) to match the formatting of other source registrations.
There was a problem hiding this comment.
@copilot open a new pull request to apply changes based on this feedback
|
@sebdraven I've opened a new pull request, #6, to work on those changes. Once the pull request is ready, I'll request review from you. |
[WIP] WIP address feedback on InfoRos domain source registration
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Ajout pour charger la base donnée de domaines inforos