Skip to content

[BlueskyBridge] fix a TypeError on null for starter pack author with no display name - #5079

Merged
dvikan merged 1 commit into
RSS-Bridge:masterfrom
clavelm:fix/bsky_TypeError
Aug 28, 2026
Merged

dvikan merged 1 commit into
RSS-Bridge:masterfrom
clavelm:fix/bsky_TypeError

Conversation

@clavelm

@clavelm clavelm commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Author of a starter pack may have no display name like falalala50 for https://bsky.app/starter-pack/falalala50.bsky.social/3mtwc5lj26427

This PR just adds a null coalescing ?? '' on the concern line.

Details
Type: TypeError
Code: 0
Message: e(): Argument #1 ($s) must be of type string, null given, called in bridges/BlueskyBridge.php on line 680
File: lib/html.php
Line: 63
Trace
#0 index.php(73): RssBridge->main()
#1 lib/RssBridge.php(39): RssBridge->{closure:RssBridge::main():37}()
#2 lib/RssBridge.php(37): BasicAuthMiddleware->__invoke()
#3 middlewares/BasicAuthMiddleware.php(13): RssBridge->{closure:RssBridge::main():37}()
#4 lib/RssBridge.php(37): CacheMiddleware->__invoke()
#5 middlewares/CacheMiddleware.php(44): RssBridge->{closure:RssBridge::main():37}()
#6 lib/RssBridge.php(37): ExceptionMiddleware->__invoke()
#7 middlewares/ExceptionMiddleware.php(17): RssBridge->{closure:RssBridge::main():37}()
#8 lib/RssBridge.php(37): SecurityMiddleware->__invoke()
#9 middlewares/SecurityMiddleware.php(19): RssBridge->{closure:RssBridge::main():37}()
#10 lib/RssBridge.php(37): MaintenanceMiddleware->__invoke()
#11 middlewares/MaintenanceMiddleware.php(10): RssBridge->{closure:RssBridge::main():37}()
#12 lib/RssBridge.php(37): TokenAuthenticationMiddleware->__invoke()
#13 middlewares/TokenAuthenticationMiddleware.php(31): RssBridge->{closure:RssBridge::main():33}()
#14 lib/RssBridge.php(34): DisplayAction->__invoke()
#15 actions/DisplayAction.php(54): DisplayAction->createResponse()
#16 actions/DisplayAction.php(89): BlueskyBridge->collectData()
#17 bridges/BlueskyBridge.php(423): BlueskyBridge->getStarterPackDescription()
#18 bridges/BlueskyBridge.php(680): e()
#19 lib/html.php(63)

@github-actions

Copy link
Copy Markdown

Pull request artifacts

Bridge Context Status
Bluesky 1 Posts from a user (current) ✔️
Bluesky 1 Posts from a user (pr) ✔️

last change: Thursday 2026-08-27 21:31:05

@dvikan
dvikan merged commit f4aa9e0 into RSS-Bridge:master Aug 28, 2026
11 checks passed
@clavelm
clavelm deleted the fix/bsky_TypeError branch August 30, 2026 12:52
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.

2 participants