Skip to content

Rtmp driver#27

Open
kyle-fitzp wants to merge 13 commits into
masterfrom
rtmp-driver
Open

Rtmp driver#27
kyle-fitzp wants to merge 13 commits into
masterfrom
rtmp-driver

Conversation

@kyle-fitzp
Copy link
Copy Markdown

@kyle-fitzp kyle-fitzp commented May 5, 2026

Specialized FFmpeg sensor driver implementation to listen for and demux RTMP streams. Note: FFmpeg's RTMP listener implementation currently does not check user credentials or the RTMP app/playpath. These features are being researched and may be added in the future.

Additionally, this PR includes some small changes to the MpegTsProcessor class. These are unlikely to break anything, but it may be worth running some quick tests on the FFmpeg sensor driver.

Testing:

Listening Test

  1. Create an RTMP driver.
  2. In Connection, leave port as 1935 (or change to any other value if the port is in use by another app).
  3. Set the host to LOCALHOST. If testing with an external RTMP source, set the host to UNSPECIFIED.
  4. Initialize and start the driver. The driver's status should state that it is listening.
  5. Start the RTMP publish. After a moment, the driver status should state that the stream has opened.
  6. Stop the RTMP publish (but leave the driver running). The driver status should state that the connection has been lost.
  7. Start the RTMP publish again. The driver status should update to state that the stream has opened.
  8. Stop the RTMP publish and driver (in either order). Verify that there is no error status in the admin panel.

Port Check Test

  1. Create an RTMP driver and start it.
  2. Create another RTMP driver with the same port and attempt to start it. The driver should not be started. A status message should identify the other RTMP driver currently using the port.

Setting up a local RTMP publisher:

With FFmpeg, run the following command in a separate terminal to start publishing RTMP:
ffmpeg -re -stream_loop -1 -i "[source video file or url]" -f flv "rtmp://localhost:1935"

@kyle-fitzp kyle-fitzp self-assigned this May 5, 2026
@kyle-fitzp kyle-fitzp linked an issue May 5, 2026 that may be closed by this pull request
Copy link
Copy Markdown

@nickgaray nickgaray left a comment

Choose a reason for hiding this comment

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

  • Add file headers
  • If you are using IntelliJ IDEA, have the AI assistant generate documentation for the classes, members, methods/functions

Copy link
Copy Markdown

@nickgaray nickgaray left a comment

Choose a reason for hiding this comment

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

GeoRobotix, Inc. is the parent company
GeoRobotix Innovative Research is not a corporation it is an LLC and subsidiary of the parent but just naming GeoRobotix Innovative Research is sufficient as it is recognized as the company.

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.

Create RTMP Driver

2 participants