Release: Version 1.20.1 (#1284)

* release: version 1.20.1

* fix: adjust the new lint rules

CHANGELOG.md

@@ -5,6 +5,21 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.20.1] - 2024-05-06
+
+### Fixed
+
+- Fix: Make TLS Probe Independent
+
+### Changed
+
+- Refactor: Separate Create Config Handler
+- CI: Upgrade actions that still use Node.js version 16
+- Refactor: Separate flag handlers
+- Test: Add tests for Monika configuration codes
+- Feat: adding sendWithCustomContent in monika-notif
+- Docs: Remove probes property from TLS checker config example
+
 ## [1.20.0] - 2024-04-18
 
 ### Added

package.json

@@ -1,7 +1,7 @@
 {
   "name": "@hyperjumptech/monika",
   "description": "Synthetic monitoring made easy",
-  "version": "1.20.0",
+  "version": "1.20.1",
   "license": "MIT",
   "author": "@hyperjumptech",
   "main": "lib/index.js",
@@ -58,7 +58,7 @@
   },
   "dependencies": {
     "@faker-js/faker": "^7.4.0",
-    "@hyperjumptech/monika-notification": "^1.17.0",
+    "@hyperjumptech/monika-notification": "^1.18.0",
     "@oclif/core": "3.16.0",
     "@oclif/plugin-help": "^6.0.9",
     "@oclif/plugin-version": "^2.0.11",

packages/notification/utils/authorization.ts

@@ -22,11 +22,11 @@
  * SOFTWARE.                                                                      *
  **********************************************************************************/
 
-import { AxiosBasicCredentials } from 'axios'
+import type { AxiosBasicCredentials } from 'axios'
 
-type BasicCredentials = AxiosBasicCredentials & {
+type BasicCredentials = {
   type: 'basic'
-}
+} & AxiosBasicCredentials
 
 type Bearer = { type: 'bearer'; token: string }
 

src/components/incident/index.ts

@@ -46,8 +46,8 @@ export function addIncident(
 export function removeIncident({
   probeID,
   probeRequestURL,
-}: Pick<Incident, 'probeID'> &
-  Partial<Pick<Incident, 'probeRequestURL'>>): void {
+}: Partial<Pick<Incident, 'probeRequestURL'>> &
+  Pick<Incident, 'probeID'>): void {
   const newIncidents = getIncidents().filter((incident) => {
     if (!probeRequestURL) {
       return probeID !== incident.probeID

src/components/probe/prober/http/request.test.ts

@@ -205,9 +205,6 @@ describe('probingHTTP', () => {
             headers.get('content-type') !== 'text/plain' ||
             body !== 'multiline string\nexample'
           ) {
-            console.error(headers.get('content-type'))
-            console.error(body)
-
             return new HttpResponse(null, {
               status: 400,
             })
@@ -248,9 +245,6 @@ describe('probingHTTP', () => {
             headers.get('content-type') !== 'text/yaml' ||
             body !== 'username: [email protected]\npassword: secret\n'
           ) {
-            console.error(headers.get('content-type'))
-            console.error(body)
-
             return new HttpResponse(null, {
               status: 400,
             })
@@ -291,9 +285,6 @@ describe('probingHTTP', () => {
             headers.get('content-type') !== 'application/xml' ||
             !body.includes('[email protected]')
           ) {
-            console.error(headers.get('content-type'))
-            console.error(JSON.stringify(body))
-
             return new HttpResponse(null, {
               status: 400,
             })
@@ -426,9 +417,6 @@ describe('probingHTTP', () => {
             headers.get('content-type') !== 'text/plain' ||
             body !== 'multiline string\nexample'
           ) {
-            console.error(headers.get('content-type'))
-            console.error(body)
-
             return new HttpResponse(null, {
               status: 400,
             })

src/plugins/metrics/prometheus/collector.test.ts

@@ -157,9 +157,7 @@ describe('Prometheus collector', () => {
   describe('Collect triggered allert metrics', () => {
     it('should not throw', () => {
       // arrange
-      const probeResult: Omit<ProbeResult, 'response'> & {
-        alertQuery: string
-      } = {
+      const probeResult = {
         probe: {
           id: '',
           name: '',
@@ -189,9 +187,7 @@ describe('Prometheus collector', () => {
 
     it('should not throw with empty request', () => {
       // arrange
-      const probeResult: Omit<ProbeResult, 'response'> & {
-        alertQuery: string
-      } = {
+      const probeResult = {
         probe: {
           id: '',
           name: '',
@@ -227,9 +223,9 @@ describe('Prometheus collector', () => {
         },
         requestIndex: 0,
         alertQuery: '',
-      } as Omit<ProbeResult, 'response'> & {
+      } as {
         alertQuery: string
-      }
+      } & Omit<ProbeResult, 'response'>
 
       // act
       const prometheusCollector = new PrometheusCollector()

src/plugins/metrics/prometheus/collector.ts

@@ -215,7 +215,7 @@ export class PrometheusCollector {
   }
 
   collectTriggeredAlert(
-    probeResult: Omit<ProbeResult, 'response'> & { alertQuery: string }
+    probeResult: { alertQuery: string } & Omit<ProbeResult, 'response'>
   ): void {
     if (!prometheusCustomCollector) {
       throw new Error('Prometheus collector is not registered')

src/utils/http.ts

@@ -22,19 +22,20 @@
  * SOFTWARE.                                                                      *
  **********************************************************************************/
 
-import axios, { AxiosRequestHeaders, AxiosResponse } from 'axios'
-import http from 'http'
-import https from 'https'
+import type { AxiosRequestHeaders, AxiosResponse } from 'axios'
+import axios from 'axios'
+import http from 'node:http'
+import https from 'node:https'
 import { Agent, type HeadersInit } from 'undici'
 
-type HttpRequestParams = Omit<RequestInit, 'headers'> & {
+type HttpRequestParams = {
   url: string
   maxRedirects?: number
   headers?: HeadersInit
   timeout?: number
   allowUnauthorizedSsl?: boolean
   responseType?: 'stream'
-}
+} & Omit<RequestInit, 'headers'>
 
 // Keep the agents alive to reduce the overhead of DNS queries and creating TCP connection.
 // More information here: https://rakshanshetty.in/nodejs-http-keep-alive/