~ofiala

Recent activity

Re: [PATCH gmni] gmnlm: treat invalid cert and cert mismatch similarly 2 months ago

From Ondřej Fiala to ~sircmpwn/gmni-devel

It didn't seem all that useful to me since all the information is already in the message and if one wishes to replace it, they can just use the 't' command. Also I haven't found a natural way to integrate it into the message.


On Jul 25, 2022 10:28, Drew DeVault <sir@cmpwn.com> wrote:
>
> We should still print the path and line number of the offending cert in 
> known_hosts here. 

[PATCH gmni v3] gmnlm: make ./ prefix mandatory for relative URLs 2 months ago

From Ondřej Fiala to ~sircmpwn/gmni-devel

Interpreting everything that isn't a command as relative URL wasn't
very intuitive and lead to accidental commands or requests being made.
This commit aims to clearly separate relative URLs from commands, which
also allows us to add basic error reporting for commands.

The only exception that had to be made was in regards to /whatever URLs
as those are impossible to support without changing the search command.
---
Forgot to commit, oops.

 src/gmnlm.c | 109 +++++++++++++++++++++++++++++++++++++---------------
 1 file changed, 77 insertions(+), 32 deletions(-)

diff --git a/src/gmnlm.c b/src/gmnlm.c
[message trimmed]

[PATCH gmni v2] gmnlm: make ./ prefix mandatory for relative URLs 2 months ago

From Ondřej Fiala to ~sircmpwn/gmni-devel

Interpreting everything that isn't a command as relative URL wasn't
very intuitive and lead to accidental commands or requests being made.
This commit aims to clearly separate relative URLs from commands, which
also allows us to add basic error reporting for commands.

The only exception that had to be made was in regards to /whatever URLs
as those are impossible to support without changing the search command.
---
Make GCC's stupid "I don't know operator precedence" warning happy

 src/gmnlm.c | 109 +++++++++++++++++++++++++++++++++++++---------------
 1 file changed, 77 insertions(+), 32 deletions(-)

diff --git a/src/gmnlm.c b/src/gmnlm.c
[message trimmed]

[PATCH gmni] gmnlm: make ./ prefix mandatory for relative URLs 2 months ago

From Ondřej Fiala to ~sircmpwn/gmni-devel

Interpreting everything that isn't a command as relative URL wasn't
very intuitive and lead to accidental commands or requests being made.
This commit aims to clearly separate relative URLs from commands, which
also allows us to add basic error reporting for commands.

The only exception that had to be made was in regards to /whatever URLs
as those are impossible to support without changing the search command.
---
 src/gmnlm.c | 109 +++++++++++++++++++++++++++++++++++++---------------
 1 file changed, 77 insertions(+), 32 deletions(-)

diff --git a/src/gmnlm.c b/src/gmnlm.c
index 884796c..87adb62 100644
--- a/src/gmnlm.c
[message trimmed]

[PATCH gmni] gmnlm: treat invalid cert and cert mismatch similarly 2 months ago

From Ondřej Fiala to ~sircmpwn/gmni-devel

gmnlm already allows overriding invalid cert error, so it makes little
sense not to allow the same for cert mismatch one. I have also reversed
the offered options for both to reflect the suggested course of action
and added a warning for invalid cert as the possibility of MitM is quite
similar to cert mismatch.
---
 src/gmnlm.c | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

diff --git a/src/gmnlm.c b/src/gmnlm.c
index 884796c..0270b07 100644
--- a/src/gmnlm.c
+++ b/src/gmnlm.c
@@ -1140,9 +1140,11 @@ tofu_callback(enum tofu_error error, const char *fingerprint,
[message trimmed]

[PATCH gmni] gmnlm: allow ~ as home alias in file:// URLs 2 months ago

From Ondřej Fiala to ~sircmpwn/gmni-devel

---
 src/gmnlm.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/src/gmnlm.c b/src/gmnlm.c
index 884796c..de5428f 100644
--- a/src/gmnlm.c
+++ b/src/gmnlm.c
@@ -425,6 +425,19 @@ do_requests(struct browser *browser, struct gemini_response *resp)
				break;
			}

			if (path[0] == '~') {
				char *home = getenv("HOME");
[message trimmed]

[PATCH gmni] gmnlm: update synopsis 2 months ago

From Ondřej Fiala to ~sircmpwn/gmni-devel

---
Sorry, completely missed the synopsis when updating the manpage last time.

 doc/gmnlm.scd | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/gmnlm.scd b/doc/gmnlm.scd
index 2c7eaef..b49cab6 100644
--- a/doc/gmnlm.scd
+++ b/doc/gmnlm.scd
@@ -6,7 +6,7 @@ gmnlm - Gemini line-mode browser

# SYNPOSIS

[message trimmed]

[PATCH gmni] gmnlm: prompt for redirects, make max redirects configurable 2 months ago

From Ondřej Fiala to ~sircmpwn/gmni-devel

Redirects are currently the most reliable way of tracking users in
geminispace. To mitigate this, similar approach as used with server
certificates is adopted -- users are prompted by default, but this
can be easily overriden with a command line flag if convenience is
preferred to tracking-resistance.
---
If this approach is too extreme, it can be changed to only prompt with
a flag, though more robust CLI might be necessary if we wish to keep
configurable redirect count along with the prompting in such case.

 doc/gmnlm.scd |  3 +++
 src/gmnlm.c   | 50 +++++++++++++++++++++++++++++++++++++++++++++-----
 2 files changed, 48 insertions(+), 5 deletions(-)
[message trimmed]

[PATCH gmni] gmnlm: tidy up help message 2 months ago

From Ondřej Fiala to ~sircmpwn/gmni-devel

* make marking of mandatory arguments consistent
* document the 'i' command
* don't document commands explained by the prompt
* unify alignment
* minor wording adjustments
---
 src/gmnlm.c | 29 +++++++++++++----------------
 1 file changed, 13 insertions(+), 16 deletions(-)

diff --git a/src/gmnlm.c b/src/gmnlm.c
index e539361..53ec416 100644
--- a/src/gmnlm.c
+++ b/src/gmnlm.c
@@ -74,22 +74,19 @@ const char *default_bookmarks =
[message trimmed]

[PATCH gmni v2] gmnlm: don't assume everything starting with b/f is b/f command 2 months ago

From Ondřej Fiala to ~sircmpwn/gmni-devel

It was impossible to enter a file:// URL because it got interpreted
as a 'f' command without any warning about invalid argument. The same
thing happened with relative addresses beginning with 'b' or 'f'.
---
Forgot to check for one case.

 src/gmnlm.c | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/gmnlm.c b/src/gmnlm.c
index 9e7b527..e539361 100644
--- a/src/gmnlm.c
+++ b/src/gmnlm.c
@@ -569,8 +569,11 @@ do_prompts(const char *prompt, struct browser *browser)
[message trimmed]