Skip to content

Add LINE Messaging API Alerter - #1724

Merged
jertel merged 5 commits into
jertel:masterfrom
nsano-rururu:line
Nov 18, 2025
Merged

Add LINE Messaging API Alerter #1724
jertel merged 5 commits into
jertel:masterfrom
nsano-rururu:line

Conversation

@nsano-rururu

@nsano-rururu nsano-rururu commented Nov 17, 2025

Copy link
Copy Markdown
Collaborator

Description

Support for LINE Messaging API, an alternative service to Line Notify, has been added.

Checklist

  • I have reviewed the contributing guidelines.
  • I have included unit tests for my changes or additions.
  • I have successfully run make test-docker with my changes.
  • I have manually tested all relevant modes of the change in this PR.
  • I have updated the documentation.
  • I have updated the changelog.

Questions or Comments

Copilot AI review requested due to automatic review settings November 17, 2025 13:24
@nsano-rururu nsano-rururu changed the title Add LINE Messaging API Add LINE Messaging API Alerter Nov 17, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request adds LINE Messaging API integration to ElastAlert2, enabling alerts to be sent to LINE applications via the LINE bot push message API.

  • Implements a new LineMessageAPIAlerter class following the existing alerter patterns
  • Adds comprehensive unit tests covering success cases, error handling, and required field validation
  • Updates schema, documentation, and loader configuration to support the new alerter

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 8 comments.

Show a summary per file
File Description
elastalert/alerters/line.py Implements the LINE Messaging API alerter with support for text message push notifications
tests/alerters/line_test.py Provides unit tests for the LINE alerter including success cases, exception handling, field validation, and multiple match scenarios
elastalert/schema.yaml Adds schema definitions for line_channel_access_token and line_to configuration options
elastalert/loaders.py Registers the LINE alerter in the alerter registry and adds the module import
docs/source/alerts.rst Documents the LINE Messaging API alerter configuration and usage examples

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread tests/alerters/line_test.py
Comment thread tests/alerters/line_test.py Outdated
Comment thread tests/alerters/line_test.py Outdated
Comment thread tests/alerters/line_test.py
Comment thread elastalert/alerters/line.py Outdated
Comment thread elastalert/alerters/line.py
Comment thread tests/alerters/line_test.py
Comment thread tests/alerters/line_test.py
nsano-rururu and others added 3 commits November 17, 2025 22:31
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@nsano-rururu

Copy link
Copy Markdown
Collaborator Author

@copilot open a new pull request to apply changes based on the comments in this thread

@nsano-rururu
nsano-rururu requested a review from jertel November 17, 2025 13:58
@jertel
jertel merged commit 417d5b4 into jertel:master Nov 18, 2025
1 of 2 checks passed
@jertel

jertel commented Nov 18, 2025

Copy link
Copy Markdown
Owner

Thanks for adding LINE support back to the app.

This relates to #1638 and #1565.

@nsano-rururu
nsano-rururu deleted the line branch November 18, 2025 14:18
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.

3 participants