- Issue created by @lkacenja
- πΊπΈUnited States lkacenja
I made sure the module works with neither core logger backend installed. With a fresh D10 installation, I installed restfullogger only. In the console I ran:
// Get the CSRF token. $.ajax({ url: '/session/token', success: (data) => {console.log(data);}, }); // Use the logger. $.ajax({ type: 'POST', url: '/dblog/logger?_format=json', headers: { 'Content-Type': 'application/json', 'X-CSRF-Token': '<insert token>', }, data: JSON.stringify({ 'message': 'Test message', 'path': '/', 'severity': 'notice', }), success: () => {console.log('yay!');}, dataType:'application/json', });
This worked as expected. The log message succeeds and fires the content off into space. I then installed and configured syslog and the same two steps resulted in the logs being sent to the designated log file.
- Status changed to Needs review
3 months ago 4:25pm 29 August 2024 - πΊπΈUnited States greggles Denver, Colorado, USA
Makes sense and LGTM. Thanks!
- Status changed to RTBC
3 months ago 4:48pm 29 August 2024 - πΊπΈUnited States rymcveigh
This looks good to me as well. The restfullogger.info.yml fail is not related.
Unable to parse modules/custom/restfullogger-3471039/restfullogger.info.yml Unexpected characters near " || ^11" at line 4 (near "core_version_requirement: ">=8" || ^11").
- πΊπΈUnited States greggles Denver, Colorado, USA
This MR could also fix the cspell soft-failure by adding a _CSPELL_WORDS line with something like:
variables: _CSPELL_WORDS: 'restfulloger, rymcveigh'
This would need the
variables:
section to be uncommented or added as well. - πΊπΈUnited States rymcveigh
This MR could also fix the cspell soft-failure by adding a _CSPELL_WORDS line
I went ahead and added the
_CSPELL_WORDS
variable in this commit. - πΊπΈUnited States greggles Denver, Colorado, USA
Excellent. Looks like there's a big fail now, but maybe that's more of an infrastructure problem?
- πΊπΈUnited States rymcveigh
I adjusted the file using the new version of the gitlab-ci template and it did the job. I'm going to go ahead and merge these changes.
-
rymcveigh β
committed 9047448d on 2.0.x authored by
lkacenja β
Issue #3471039 by lkacenja: Unnecessary dependency on Database Logging...
-
rymcveigh β
committed 9047448d on 2.0.x authored by
lkacenja β
- Status changed to Fixed
3 months ago 10:26pm 29 August 2024 - πΊπΈUnited States greggles Denver, Colorado, USA
Nice catch! I was confused by the fails yesterday. Glad you figured that out.
And with these changes the pipelines are all so very happy https://git.drupalcode.org/project/restfullogger/-/pipelines/268656
Automatically closed - issue fixed for 2 weeks with no activity.