lz233 / Tarnhelm

The magic to clean sharing links up.
https://tarnhelm.project.ac.cn
GNU General Public License v3.0
422 stars 17 forks source link

add redirect rule #40

Closed Bluemangoo closed 1 year ago

Bluemangoo commented 1 year ago

Resolves #10

添加重定向规则,对于符合的域名会先发一个请求查看是否为重定向,如果是则用重定向url替换后处理其它规则。

目前仍然存在的问题

(第一次写安卓和 Kotlin,写得不好请见谅)

截图

image image image

Bluemangoo commented 1 year ago

Test app with package name cn.ac.lz233.tarnhelm.test, signed by bluemangoo:

test redirect rule:

tarnhelm://rule?redirect=eyJhIjoiQuermSIsImUiOiJiMjMudHYiLCJkIjoiQmx1ZW1hbmdvbyJ9

test para rule:

tarnhelm://rule?parameter=eyJhIjoiQuermSIsImUiOiJ3d3cuYmlsaWJpbGkuY29tIiwiZiI6MCwiZyI6WyIiXSwiZCI6IkJsdWVtYW5nb28ifQ%3D%3D
Bluemangoo commented 1 year ago

Weekly ping

lz233 commented 1 year ago

在思考要不要加这个功能,因为其实有悖于这个app的设计思路,和不知道什么时候会摸出来的扩展功能。

Bluemangoo commented 1 year ago

确实我写得不太爽,网络请求在主线程感觉很难受。如果处理逻辑在新线程运行应该好一点。

Bluemangoo commented 1 year ago

只能说这个功能还是比较急切的。当然我也可以考虑重新造一个app来处理这类链接,那样就很麻烦。我往这里提pr只不过是因为这样工作量最小罢了。

lz233 commented 1 year ago

我想我们可以在现有的规则里加入重定向的选项,而不是单独添加一个分类。 如果这个短链接的域名被规则匹配到了且这个规则的重定向选项打开了,那我们先对匹配到的链接进行重定向处理。

Bluemangoo commented 1 year ago

问题是,你怎么确定重定向终点是哪个域名。

我现在的处理是在规则处理之前加一步处理重定向。

image

lz233 commented 1 year ago

有道理,刚才我只考虑到了b站,但是tarnhelm是支持匹配多个规则的,只需要把重定向的规则放在前面就行

Bluemangoo commented 1 year ago

另外我在处理闪退的问题,我想知道在哪里可以检查并初始化这个表。

在缺表的时候会报 java.lang.IllegalStateException,如果你在现有规则上追加也要考虑这个问题。

Bluemangoo commented 1 year ago

另外,你还要考虑下正则规则。

正则规则不能这么简单粗暴匹配吧好像。

lz233 commented 1 year ago

@Bluemangoo https://developer.android.com/training/data-storage/room/migrating-db-versions 一直懒得做

Bluemangoo commented 1 year ago

@Bluemangoo https://developer.android.com/training/data-storage/room/migrating-db-versions 一直懒得做

thx

lz233 commented 1 year ago

总之,给我几天时间让我思考一下逻辑,今天肯定没时间(

Bluemangoo commented 1 year ago

yep, i'll fix the bug then

Bluemangoo commented 1 year ago

fixed.

Bluemangoo commented 1 year ago

我能做的都做完了,剩下看你了。

Bluemangoo commented 1 year ago

Weekly ping - (1)

Bluemangoo commented 1 year ago

Weekly ping @lz233 - (2)

lz233 commented 1 year ago

@Bluemangoo 忙(