Changeset 7677448 in mod_gnutls


Ignore:
Timestamp:
Aug 13, 2021, 4:25:50 PM (2 months ago)
Author:
GitHub <noreply@…>
Branches:
master
Children:
b6ce8ad
Parents:
e809a17a (diff), 272833c (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.
git-author:
Airtower <fiona.klute@…> (08/13/21 16:25:50)
git-committer:
GitHub <noreply@…> (08/13/21 16:25:50)
Message:

Merge pull request #6 from airtower-luna/log-levels

Adjust log levels for connection errors

Files:
5 edited

Legend:

Unmodified
Added
Removed
  • .github/workflows/analysis.yaml

    re809a17a r7677448  
    1313      - '!doc/**'
    1414      - '!**.md'
     15
     16permissions:
     17  contents: read
     18  security-events: write
    1519
    1620jobs:
  • .github/workflows/build.yaml

    re809a17a r7677448  
    88      - 'debian/**'
    99      - 'pristine-tar'
     10
     11permissions:
     12  contents: read
     13
    1014jobs:
    1115  build:
  • .github/workflows/valgrind.yaml

    re809a17a r7677448  
    1212      - '!doc/**'
    1313      - '!**.md'
     14
     15permissions:
     16  contents: read
     17
    1418jobs:
    1519  debian-buster:
  • src/gnutls_hooks.c

    re809a17a r7677448  
    11001100    /* check if session needs closing */
    11011101    mgs_handle_t *ctxt = (mgs_handle_t *) data;
    1102     if (ctxt->session != NULL)
     1102    if (ctxt->session == NULL)
     1103        return APR_SUCCESS;
     1104
     1105    if (ctxt->c->aborted)
     1106    {
     1107        ap_log_cerror(APLOG_MARK, APLOG_DEBUG, APR_SUCCESS, ctxt->c,
     1108                      "%s: TLS %sconnection aborted, cleaning up.",
     1109                      __func__, IS_PROXY_STR(ctxt));
     1110    }
     1111    else
    11031112    {
    11041113        ap_log_cerror(APLOG_MARK, APLOG_WARNING, APR_ECONNABORTED, ctxt->c,
     
    11121121        while (ret == GNUTLS_E_INTERRUPTED || ret == GNUTLS_E_AGAIN);
    11131122        if (ret != GNUTLS_E_SUCCESS)
    1114             ap_log_cerror(APLOG_MARK, APLOG_INFO, APR_EGENERAL, ctxt->c,
     1123            ap_log_cerror(APLOG_MARK, APLOG_DEBUG, APR_EGENERAL, ctxt->c,
    11151124                          "%s: error while closing TLS %sconnection: %s (%d)",
    11161125                          __func__, IS_PROXY_STR(ctxt),
     
    11201129                          "%s: TLS %sconnection closed.",
    11211130                          __func__, IS_PROXY_STR(ctxt));
    1122         /* De-Initialize Session */
    1123         gnutls_deinit(ctxt->session);
    1124         ctxt->session = NULL;
    1125     }
     1131    }
     1132
     1133    /* De-Initialize Session */
     1134    gnutls_deinit(ctxt->session);
     1135    ctxt->session = NULL;
    11261136    return APR_SUCCESS;
    11271137}
  • src/gnutls_io.c

    re809a17a r7677448  
    254254            } else if (rc == GNUTLS_E_REHANDSHAKE) {
    255255                /* A client has asked for a new Hankshake. Currently, we don't do it */
    256                 ap_log_cerror(APLOG_MARK, APLOG_INFO,
     256                ap_log_cerror(APLOG_MARK, APLOG_DEBUG,
    257257                        ctxt->input_rc,
    258258                        ctxt->c,
     
    262262            } else if (rc == GNUTLS_E_WARNING_ALERT_RECEIVED) {
    263263                rc = gnutls_alert_get(ctxt->session);
    264                 ap_log_cerror(APLOG_MARK, APLOG_INFO,
     264                ap_log_cerror(APLOG_MARK, APLOG_DEBUG,
    265265                        ctxt->input_rc,
    266266                        ctxt->c,
     
    270270            } else if (rc == GNUTLS_E_FATAL_ALERT_RECEIVED) {
    271271                rc = gnutls_alert_get(ctxt->session);
    272                 ap_log_cerror(APLOG_MARK, APLOG_INFO,
     272                ap_log_cerror(APLOG_MARK, APLOG_DEBUG,
    273273                        ctxt->input_rc,
    274274                        ctxt->c,
     
    281281                /* Some Other Error. Report it. Die. */
    282282                if (gnutls_error_is_fatal(rc)) {
    283                     ap_log_cerror(APLOG_MARK,
    284                             APLOG_INFO,
    285                             ctxt->input_rc,
    286                             ctxt->c,
    287                             "GnuTLS: Error reading data. (%d) '%s'",
    288                             rc,
    289                             gnutls_strerror(rc));
     283                    ap_log_cerror(
     284                        APLOG_MARK, APLOG_DEBUG,
     285                        ctxt->input_rc,
     286                        ctxt->c,
     287                        "GnuTLS: Error reading data. (%d) '%s'",
     288                        rc, gnutls_strerror(rc));
    290289                } else if (*len > 0) {
    291290                    ctxt->input_rc = APR_SUCCESS;
     
    778777                if (ret < 0) {
    779778                    /* error sending output */
    780                     ap_log_cerror(APLOG_MARK, APLOG_INFO, ctxt->output_rc,
     779                    ap_log_cerror(APLOG_MARK, APLOG_DEBUG, ctxt->output_rc,
    781780                                  ctxt->c,
    782781                                  "GnuTLS: Error writing data. (%d) '%s'",
Note: See TracChangeset for help on using the changeset viewer.