Skip to content

new page entrance + new dedicated dir for bridges + some fixe, optimizat...#2

Closed
Draeli wants to merge 1 commit intoRSS-Bridge:masterfrom
Draeli:new_feature
Closed

new page entrance + new dedicated dir for bridges + some fixe, optimizat...#2
Draeli wants to merge 1 commit intoRSS-Bridge:masterfrom
Draeli:new_feature

Conversation

@Draeli
Copy link
Contributor

@Draeli Draeli commented Aug 9, 2013

  • All bridge move to dedicated dir.
  • New page entrance for list all disponible bridge and provide an direct request with bridge functionnality
  • Break bridges dependance to call lib, page entrance do this call on demand (not yet documentation, soon ...)
  • Bridge come now with annotation (see directly bridge example for example)
  • Some optimizations and fix
  • Request can be realized in POST, GET (before only GET) and both
  • Due to request change, cache modification to create specific for each demand (test are not full, I think I miss something)

Annotation for bridge : page entrance can list but need some informations, that informations are give by annotation system and depend of this list of parameters :

  • @name (required) Human bridge name
  • @description (optional) Bridge description
  • @useN N is integer between 1 and infinite. One use is need for each rss bridge functionnality
    Exemple 1 : @use1(q="keyword search")
    In this case bridge can be use with one parameter and page entrance is going to ask for one input with "keyword search" as description
    Exemple 2 : @use2(q="keyword search", other="another parameter")

@Draeli Draeli closed this Aug 9, 2013
@Draeli Draeli deleted the new_feature branch August 9, 2013 16:46
logmanoriginal pushed a commit that referenced this pull request Oct 2, 2016
This was referenced Jul 4, 2019
dvikan pushed a commit that referenced this pull request Jun 10, 2023
* Update NyaaTorrentsBridge.php

* lint

* lint #2

* Sir Lint the Third

* Add torrent id to custom fields

* Proposed improvements
dvikan added a commit that referenced this pull request Jan 23, 2024
8192: explode(): Passing null to parameter #2 ($string) of type string is deprecated in bridges/SpotifyBridge.php line 322
@nuttertkk nuttertkk mentioned this pull request Nov 2, 2025
Mynacol added a commit that referenced this pull request Jan 15, 2026
Won't work without API keys anymore. The change is code-wise easy to do, while scraping the HTML would be cumbersome.

 Bridge returned error 401! (20449)
2025-12-27 03:26:04

https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/401
Details
Type: HttpException
Code: 401
Message: https://gelbooru.com/index.php?&page=dapi&s=post&q=index&json=1&pid=0&limit=&tags= resulted in 401 Unauthorized
File: lib/http.php
Line: 34
Trace
#0 index.php(73): RssBridge->main()
#1 lib/RssBridge.php(39): RssBridge->{closure}()
#2 lib/RssBridge.php(37): BasicAuthMiddleware->__invoke()
#3 middlewares/BasicAuthMiddleware.php(13): RssBridge->{closure}()
#4 lib/RssBridge.php(37): CacheMiddleware->__invoke()
#5 middlewares/CacheMiddleware.php(44): RssBridge->{closure}()
#6 lib/RssBridge.php(37): ExceptionMiddleware->__invoke()
#7 middlewares/ExceptionMiddleware.php(17): RssBridge->{closure}()
#8 lib/RssBridge.php(37): SecurityMiddleware->__invoke()
#9 middlewares/SecurityMiddleware.php(19): RssBridge->{closure}()
#10 lib/RssBridge.php(37): MaintenanceMiddleware->__invoke()
#11 middlewares/MaintenanceMiddleware.php(10): RssBridge->{closure}()
#12 lib/RssBridge.php(37): TokenAuthenticationMiddleware->__invoke()
#13 middlewares/TokenAuthenticationMiddleware.php(10): RssBridge->{closure}()
#14 lib/RssBridge.php(34): DisplayAction->__invoke()
#15 actions/DisplayAction.php(54): DisplayAction->createResponse()
#16 actions/DisplayAction.php(89): GelbooruBridge->collectData()
#17 bridges/GelbooruBridge.php(80): getContents()
#18 lib/contents.php(131): HttpException::fromResponse()
#19 lib/http.php(34)

Context
Query: action=display&bridge=GelbooruBridge&p=0&t=&l=&format=Html
Version: 2025-08-05
OS: Linux
PHP: 8.2.29


Co-authored-by: Mynacol <git@mynacol.xyz>
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