98b7eff...
by
Vadim Peretokin <email address hidden>
Add: automatically update map on all changes via API (#7187)
<!-- Keep the title short & concise so anyone non-technical can
understand it,
the title appears in PTB changelogs -->
#### Brief overview of PR changes/additions
Automatically update map on all map changes via API while not losing
performance.
#### Motivation for adding to Mudlet
Better user experience.
#### Other info (issues closed, discussion etc)
Alleviates the need for calling https://wiki.mudlet.org/w/Manual:Mapper_Functions#updateMap and fixes
the inconsistency where some functions were updating the map
automatically and others haven't.
#### Motivation for adding to Mudlet
Builds were failing, but not every time, like https://ci.appveyor.com/project/Mudlet/mudlet/builds/49542655
Last lines were
```
==== compiling and installing mingw-get ====
---- Downloading ----
Exception calling "DownloadFile" with "2" argument(s): "The underlying connection was closed: Could not establish
trust relationship for the SSL/TLS secure channel."
```
I looked in the script files and found URL associated with mingw-get and
downloaded it myself a few times. It uses a 302 Location redirect going
to 2 different URLs, sometimes working and others with an expired
certificate.
Then I found the link above which is similar URL but different `m=`.
Looks like `m` is for mirror, and the old one is name of a German
university that maybe is no longer mirroring for them or something.
<!-- Keep the title short & concise so anyone non-technical can
understand it,
the title appears in PTB changelogs -->
#### Brief overview of PR changes/additions
Add the GMCP authenticate spec and client credentials: https://wiki.mudlet.org/w/Standards:GMCP_Authentication
#### Motivation for adding to Mudlet
To solve the issues we have in trying to login to games - not every game
accepts "character\npassword" kind of login - while not getting into a
security quagmire that comes with exposing passwords to the Lua
subsystem.
#### Other info (issues closed, discussion etc)
---------
Co-authored-by: Vadim Peretokin <email address hidden>
Co-authored-by: Marco Fontani <email address hidden>
Co-authored-by: Stephen Lyons <email address hidden>
598ec5a...
by
"dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com>
Infrastructure: Bump ZedThree/clang-tidy-review from 0.17.2 to 0.18.0 (#7196)
Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.
<details>
<summary>Dependabot commands and options</summary>
<br />
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after
your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating
it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)
Calling setAreaName() before the echo resulted in the wrong area name
being sent to the user. Call setAreaName() after instead.
<!-- Keep the title short & concise so anyone non-technical can
understand it,
the title appears in PTB changelogs -->
#### Brief overview of PR changes/additions
Using 'area rename' command would show
`> area rename New Area Name`
`(mapper): Renamed New Area Name to New Area Name (5).`
now displays correctly
`(mapper): Renamed Old Area Name to New Area Name (5).`
#### Motivation for adding to Mudlet
Improve user experience.
#### Other info (issues closed, discussion etc)
Fix will apply to IRE mapping script as well.
97ac27c...
by
"dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com>
Infrastructure: Bump ZedThree/clang-tidy-review from 0.17.1 to 0.17.2 (#7192)
Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.
<details>
<summary>Dependabot commands and options</summary>
<br />
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after
your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating
it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)
Infrastructure: remove some unused variables left behind after a recent PR (#7180)
#### Brief overview of PR changes/additions
Remove some unused local variables left over from a recent PR.
#### Motivation for adding to Mudlet
These generate warnings from some code analysers (e.g. clang-tidy) that
might swamp more important warnings that we do need to worry about. Some
of the variable declarations do not even include an initialisation which
is something that analysers also get unhappy about.
#### Other info (issues closed, discussion etc)
It looks like this came about in #7167.
Signed-off-by: Stephen Lyons <email address hidden>
Infrastructure: reposition const before type specifier (#7179)
#### Brief overview of PR changes/additions
This PR tries to put all `const`s before the type (class).
#### Motivation for adding to Mudlet
There is a mix of positioning of `const`s where it is used to indicate
that a variable is not to be modified by program code, we tend to put it
before the type but a prior PR (which looks to have been done with an
automated tool) has resulted in a mix of cases some with the const
adjacent to the variable. This lack of consistency can be confusing.
#### Other info (issues closed, discussion etc)
It looks like this came about in #6843.
This will upset the Danger detector because of the number of files
modified but it should be fairly straightforward to review.
Signed-off-by: Stephen Lyons <email address hidden>
Improve: add controlCharacterHandling to Lua get- & set-config (#7176)
#### Brief overview of PR changes/additions
Extend `TLuaInterpreter::setConfig(...)` and
`TLuaInterpreter::getConfig(...)` for an additional case:
`controlCharacterHandling` and also add that "key" to the list in the
external `./mudlet-lua/lua/Other.lua` script file so the no argument
`getConfig()` case includes it as well.
#### Motivation for adding to Mudlet
Medievia MUD will want this to enable a package to configure this to
`"oem"` (out of the other `"asis"` and `"picture"` options).
#### Other info (issues closed, discussion etc)
Also rename a couple of variables in the section of `"caretShortcut"`
for the `setConfig(...)` case as it was using `key`/`keys` for things
that are `value`/`values`!
Also improve a couple of other setConfig cases (`caretShortcut` and
`commandLineHistorySaveSize`) which take one of a number of values so
that entering an invalid string produces a `nil` + error message rather
than throwing an error - as is our practice for other run-time value
(rather than type) problems.
Signed-off-by: Stephen Lyons <email address hidden>
