Fix spelling issues detected by codespell

Closes cockpit-project#14282

containers/bastion/cockpit-auth-ssh-key

@@ -31,7 +31,7 @@ COCKPIT_SSH_COMMAND = "/usr/libexec/cockpit-ssh"
 # we will try to decrypt the key with the given password. If successful
 # we send the decrypted key to cockpit-ws for use with cockpit-ssh.
 # Once finished we exec cockpit-ssh to actually establish the ssh connection.
-# All comunication with cockpit-ws happens on stdin and stdout using the
+# All communication with cockpit-ws happens on stdin and stdout using the
 # cockpit protocol
 # (https://github.com/cockpit-project/cockpit/blob/master/doc/protocol.md)
 

doc/authentication.md

@@ -29,7 +29,7 @@ Authentication commands are called with a single argument which is the host that
 is connecting to. They communicate with their parent process using the cockpit protocol on
 stdin and stdout.
 
-Credentials can then be retrived by issuing a authorize command with a challenge. The challenge
+Credentials can then be retrieved by issuing a authorize command with a challenge. The challenge
 should correspond to the authorization type in header (ei: Basic or Bearer). For example:
 
 

doc/guide/feature-virtualmachines.xml

@@ -30,7 +30,7 @@
     <para>
         In order to manipulate as unprivileged user VMs on system connection through
         <emphasis>cockpit-machines</emphasis> plugin using libvirt D-Bus provider you need to follow
-        the intructions specified <ulink url="https://libvirt.org/dbus.html#usage">here</ulink>.
+        the instructions specified <ulink url="https://libvirt.org/dbus.html#usage">here</ulink>.
     </para>
     </section>
 

doc/guide/gtk-doc.xsl

@@ -68,7 +68,7 @@
   <!-- ========================================================= -->
 
   <!-- l10n is slow, we don't ue it, so we'd like to turn it off
-       this atleast avoid the re-evaluation -->
+       this at least avoid the re-evaluation -->
   <xsl:template name="l10n.language">en</xsl:template>
 
   <xsl:param name="gtkdoc.l10n.xml" select="document('http://docbook.sourceforge.net/release/xsl/current/common/en.xml')"/>
@@ -657,7 +657,7 @@ Get a newer version at http://docbook.sourceforge.net/projects/xsl/
           </xsl:when>
           <!-- this is not yet very nice, as it requires all glossdic/indexdiv
           elements having a anchor element. maybe we can customize the xsl
-          to automaticaly create local anchors
+          to automatically create local anchors
           -->
           <xsl:when test="count($glssections) > 0">
             <tr>

doc/guide/packages.xml

@@ -90,8 +90,8 @@ $ cockpit-bridge --packages
           <para>By default Cockpit serves packages using a strict
             <ulink url="https://en.wikipedia.org/wiki/Content_Security_Policy">Content Security Policy</ulink>,
             which among other things does not allow inline styles or scripts. This can
-            be overriden on a per-package basis, with this setting.</para>
-          <para>If the overriden content security policy does not contain a <code>default-src</code>,
+            be overridden on a per-package basis, with this setting.</para>
+          <para>If the overridden content security policy does not contain a <code>default-src</code>,
             <code>connect-src</code>, <code>base-uri</code>, <code>form-action</code>, <code>object-src</code>,
             or <code>block-all-mixed-content</code> then these will be added to the policy from the manifest.</para>
         </listitem>

doc/man/pam_ssh_add.xml

@@ -50,7 +50,7 @@
   <para>
     By default the session module will start a new ssh-agent and run
     ssh-add, loading any keys that exist in the default path for the
-    newly logged in user. If any keys prompt for a password, and a authenication
+    newly logged in user. If any keys prompt for a password, and a authentication
     token was successfully stored, that token will be provided as the password.
   </para>
 

doc/protocol.md

@@ -223,7 +223,7 @@ by adding a "auth-method-results" object to the close object. This is mainly
 useful to provide helpful error messages and suggestions to users.
 
 The "auth-method-results" object contains a key for each method that cockpit-ws
-is able to atempt authentication with as well as the result of the atempt.
+is able to attempt authentication with as well as the result of the attempt.
 For example:
 
     {

doc/routing.md

@@ -5,7 +5,7 @@ Flow Control
 
 Cockpit's protocol passes messages on reliable, ordered underlying
 transports (eg: WebSockets, HTTP responses, stdio) between multiple
-peers, aranged in in a non-rooted tree graph.
+peers, arranged in in a non-rooted tree graph.
 
 Cockpit's flow control is not about replaying messages, but avoiding
 flooding any peer by sending messages along any link in the graph
@@ -19,7 +19,7 @@ down a certain channel which is flooding communication.
 Therefore we use a windowing flow control protocol. It is based around
 per-channel "ping" control messages. In the Cockpit protocol "ping"
 messages in a channel are responded to by specific leaves of the graph.
-They are replied to with a "pong" contaning otherwise identical content.
+They are replied to with a "pong" containing otherwise identical content.
 Other "ping" messages are used as keep alives, but these are beyond the
 scope of this description.
 

examples/poc-cim/test.html

@@ -60,7 +60,7 @@ <h2>Proof of Concept: CIM Access</h2>
 
             /* Wheee */
             $("table").show();
-            message("Done! " + instances.length + " processes, " + data.length + " bytes transfered from CIM ");
+            message("Done! " + instances.length + " processes, " + data.length + " bytes transferred from CIM ");
 
             $(response).find("INSTANCE").each(function() {
                 tbody.append(

pkg/docker/docker.css

@@ -62,8 +62,8 @@ div.spinner {
 }
 
 /*
- * Used to make a table cell that containts buttons have less padding
- * HACK: Because CSS has no parant selector.
+ * Reduce padding on table cells that contain buttons
+ * HACK: Because CSS has no parent selector.
  */
 .cell-buttons,
 .table > tbody > tr > .cell-buttons {

pkg/docker/docker.js

@@ -368,7 +368,7 @@ docker.console = function console_(container_id, command, options) {
     function prepare() {
         self.connected = false;
 
-        /* Nothing to prepare? Connect diretly to container? */
+        /* Nothing to prepare? Connect directly to container. */
         if (!exec) {
             if (tty === false) {
                 return attach("GET /v1.15/containers/" + encodeURIComponent(container_id) +
@@ -508,7 +508,7 @@ docker.console = function console_(container_id, command, options) {
         }
     };
 
-    /* Allows the curser to restart the attach request */
+    /* Allows the cursor to restart the attach request */
     self.connect = function connect() {
         self.close("disconnected");
         prepare();

pkg/kdump/kdump-client.js

@@ -114,7 +114,7 @@ export class KdumpClient {
             return dfd.promise();
         } else if (target.target === "nfs") {
             if (!target.nfs.value.match("\\S+:/.+"))
-                dfd.reject(_("nfs dump target isn't formated as server:path"));
+                dfd.reject(_("nfs dump target isn't formatted as server:path"));
         } else if (target.target === "ssh") {
             if (!target.ssh.value.trim())
                 dfd.reject(_("ssh server is empty"));

pkg/lib/journal.js

@@ -65,7 +65,7 @@ export var journal = { };
  *  .done(function(entries) { }): Called when done, @entries is
  *         an array of all journal entries loaded. If .stream()
  *         has been invoked then @entries will be empty.
- *  .fail(funciton(ex) { }): called if the operation fails
+ *  .fail(function(ex) { }): called if the operation fails
  *  .stream(function(entries) { }): called when we receive entries
  *         entries. Called once per batch of journal @entries,
  *         whether following or not.

pkg/lib/machine-dialogs.js

@@ -149,7 +149,7 @@ function Dialog(selector, address, machines_ins, codes) {
 
         var machine = self.machines_ins.lookup(address);
         if (machine && machine.host_key && !machine.on_disk) {
-            conn_options['temp-session'] = false; /* Compatiblity option */
+            conn_options['temp-session'] = false; /* Compatibility option */
             conn_options.session = 'shared';
             conn_options['host-key'] = machine.host_key;
         }
@@ -901,7 +901,7 @@ function SyncUsers(dialog) {
 
         dialog.run(promise);
 
-        /* A successfull sync should close the dialog */
+        /* A successful sync should close the dialog */
         dialog.set_on_success(null);
 
         promise.always(function () {
@@ -965,7 +965,7 @@ function SyncUsers(dialog) {
     }
 
     function render() {
-        function formated_groups() {
+        function formatted_groups() {
             if (this.groups)
                 return this.groups.join(", ");
         }
@@ -986,7 +986,7 @@ function SyncUsers(dialog) {
             users : user_list,
             perm_failed : perm_failed ? cockpit.message(perm_failed) : null,
             allows_password : allows_password,
-            formated_groups: formated_groups,
+            formatted_groups: formatted_groups,
         });
 
         dialog.get_sel(".modal-content").html(text);

pkg/lib/machine-sync-users.html

@@ -36,7 +36,7 @@ <h4 translate="yes" class="pf-c-alert__title">Error loading users: {{perm_failed
                   <td>{{username}}</td>
                   <td>{{name}}</td>
                   <td>&#8226;&#8226;&#8226;&#8226;&#8226;</td>
-                  <td>{{formated_groups}}</td>
+                  <td>{{formatted_groups}}</td>
               </tr>
         {{/users}}
         </tbody>

pkg/lib/page.scss

@@ -202,7 +202,7 @@ input[type=number] {
 }
 
 @-moz-document url-prefix() {
-    /* Accomodate Firefox styling selects with slightly different padding. */
+    /* Accommodate Firefox styling selects with slightly different padding. */
     .ct-select {
         padding-left: 0.25em;
     }

pkg/lib/patterns.js

@@ -237,7 +237,7 @@ window.addEventListener("hashchange", function() {
  *
  * The following div.slider-bar if present is resized to fill the remainder
  * of the slider if not given a specific size. You can put more div.slider-bar
- * inside it to reflect squashing other prevous allocations.
+ * inside it to reflect squashing other previous allocations.
  *
  * If the following div.slider-bar have a width specified, then the
  * slider supports the concept of overflowing. If the slider overflows

pkg/lib/service.js

@@ -41,8 +41,8 @@ import cockpit from "cockpit";
  * Either 'undefined' when the value can't be retrieved, or a
  * boolean that tells whether the service is started 'enabled'.
  * What it means exactly for a service to be enabled depends on
- * the service, but a enabled service is usually started on boot,
- * no matter wether other services need it or not.  A disabled
+ * the service, but an enabled service is usually started on boot,
+ * no matter whether other services need it or not.  A disabled
  * service is usually only started when it is needed by some other
  * service.
  *

pkg/machines/components/networks/utils.js

@@ -40,7 +40,7 @@ export function validateIpv4(address) {
 }
 
 /**
- * validates correctnes of ipv4 prefix length or mask
+ * validates correctness of ipv4 prefix length or mask
  *
  * @param {string} prefixOrNetmask
  * @returns {boolean}
@@ -167,7 +167,7 @@ export function validateIpv6(address) {
 }
 
 /**
- * validates correctnes of ipv6 prefix length
+ * validates correctness of ipv6 prefix length
  *
  * @param {string} prefixOrNetmask
  * @returns {boolean}

pkg/machines/components/storagePools/createStoragePoolDialog.jsx

@@ -71,7 +71,7 @@ const StoragePoolTypeRow = ({ onValueChanged, dialogValues, libvirtVersion }) =>
         poolTypes.push({ type: 'iscsi-direct', detail: _("iSCSI direct Target") });
 
     /* TODO
-        { type: 'fs', detail _("Pre-formated Block Device") },
+        { type: 'fs', detail _("Pre-formatted Block Device") },
         { type: 'gluster', detail _("Gluster Filesystem") },
         { type: 'mpath', detail _("Multipath Device Enumerator") },
         { type: 'rbd', detail _("RADOS Block Device/Ceph") },

pkg/machines/components/vmOverviewTabLibvirt.jsx

@@ -233,7 +233,7 @@ class VmOverviewTabLibvirt extends React.Component {
                         );
                     } else {
                         firmwareLinkWrapper = (
-                            <Tooltip id='firmware-edit-disabled-on-transient' content={_("Transient VMs don't support editting firmware configuration")}>
+                            <Tooltip id='firmware-edit-disabled-on-transient' content={_("Transient VMs don't support editing firmware configuration")}>
                                 {firmwareLink(true)}
                             </Tooltip>
                         );

pkg/machines/components/vmnetworktab.jsx

@@ -161,7 +161,7 @@ class VmNetworkTab extends React.Component {
                     const ips = (iface && iface[2]) ? iface[2] : undefined;
 
                     if (!ips) {
-                    // There is not IP address assosiated with this NIC
+                    // There is not IP address associated with this NIC
                         return _("Unknown");
                     } else {
                         return (

pkg/machines/helpers.js

@@ -512,7 +512,7 @@ export function getBootOrderDevices(vm) {
     }
 
     // if boot order was defined in os->boot (old way), array contains only devices without boot order
-    // in case of boot order devined in devices->boot (new way), array contains all devices
+    // in case of boot order defined in devices->boot (new way), array contains all devices
     for (let i = 0; i < disks.length; i++) {
         const disk = disks[i];
 
@@ -524,7 +524,7 @@ export function getBootOrderDevices(vm) {
     }
 
     // if boot order was defined in os->boot (old way), array contains only devices without boot order
-    // in case of boot order devined in devices->boot (new way), array contains all devices
+    // in case of boot order defined in devices->boot (new way), array contains all devices
     for (let i = 0; i < ifaces.length; i++) {
         const iface = ifaces[i];
 

pkg/machines/libvirt-dbus.js

@@ -1023,7 +1023,7 @@ function doUsagePolling(name, connectionName, objPath) {
 }
 
 /**
- * Subscribe to D-Bus signals and defines the handlers to be invoked in each occassion.
+ * Subscribe to D-Bus signals and defines the handlers to be invoked in each occasion.
  * @param  {String} connectionName D-Bus connection type; one of session/system.
  * @param  {String} libvirtServiceName
  */

pkg/networkmanager/interfaces.js

@@ -191,7 +191,7 @@ function NetworkManagerModel() {
      * structure.  This has the advantage of avoiding wasting
      * resources for maintaining the unused proxies, avoids some code
      * complexity, and allows to do the right thing with the
-     * pecularities of the NetworkManager API.
+     * peculiarities of the NetworkManager API.
      *
      * However, we do use a fake object manager since that allows us
      * to avoid a lot of 'GetAll' round trips during initialization
@@ -2021,7 +2021,7 @@ function choice_title(choices, choice, def) {
  * 2) Make the change
  * 3) Destroy the checkpoint
  *
- * If step 2 breaks the connection, step 3 wont happen and the
+ * If step 2 breaks the connection, step 3 won't happen and the
  * checkpoint will roll back after some time.  This is supposed to
  * restore connectivity, so steps 2 and 3 will complete at that time,
  * and step 3 will fail because the checkpoint doesn't exist anymore.
@@ -2068,7 +2068,7 @@ function choice_title(choices, choice, def) {
  * changes, and it is thus important to show it if at all possible.
  */
 
-/* Considerations for chosing the times below
+/* Considerations for choosing the times below
  *
  * curtain_time too short:  Curtain comes up too often for good changes.
  *
@@ -3548,7 +3548,7 @@ function set_slave(model, master_connection, master_settings, slave_type,
         }
 
         return settings_applier(model, iface.Device, main_connection)(slave_settings).then(function () {
-            // If the master already exists, activate or deactivate the slave immediatly so that
+            // If the master already exists, activate or deactivate the slave immediately so that
             // the settings actually apply and the interface becomes a slave.  Otherwise we
             // activate it later when the master is created.
             if (master_connection) {
@@ -4667,7 +4667,7 @@ function PageNetworkMacSettings() {
  * pages and dialogs, and expects page.setup, page.enter, page.show,
  * and page.leave to be called at the right times.
  *
- * We cater to this with a little compatability shim consisting of
+ * We cater to this with a little compatibility shim consisting of
  * 'dialog_setup', 'page_show', and 'page_hide'.
  */
 

pkg/playground/jquery-patterns.js

@@ -22,7 +22,7 @@ $(function() {
     });
 
     /*
-     * These are failures targetted at specific fields. Note that we set
+     * These are failures targeted at specific fields. Note that we set
      * selectors on the target property of the exception. Also note how
      * the .dialog('failure') accepts multiple exceptions as arguments.
      */

pkg/playground/test.html

@@ -17,7 +17,7 @@
         <br/>
         <br/>
         <div class="cockpit-internal-reauthorize">
-            <button class="pf-c-button pf-m-secondary">Priveleged Action</button>
+            <button class="pf-c-button pf-m-secondary">Privileged Action</button>
             <span></span>
         </div>
         <div class="super-channel">

pkg/playground/translate.html

@@ -64,7 +64,7 @@ <h2>Old Glade style</h2>
         <p>Note that we do <b>not support</b>:<p>
         <ul>
             <li>Interpolation of variables.</li>
-            <li>Tranlatable attributes.</li>
+            <li>Translatable attributes.</li>
             <li>Pluralization</li>
             <li>The <code>&lt;translate&gt;</code> element</li>
         </ul>

pkg/shell/nav.jsx

@@ -157,7 +157,7 @@ function PageStatus({ status, name }) {
     );
 }
 
-function FormatedText({ keyword, term }) {
+function FormattedText({ keyword, term }) {
     function split_text(text, term) {
         const b = text.toLowerCase().indexOf(term);
         const e = b + term.length;
@@ -194,9 +194,9 @@ export function CockpitNavItem(props) {
         <li className={classes.join(" ")}>
             <span className={"pf-c-nav__link" + (props.active ? " pf-m-current" : "")} data-for={props.to}>
                 <a href={props.to}>
-                    { props.header && <span className="hint">{header_matches ? <FormatedText keyword={props.header} term={props.term} /> : props.header}</span> }
-                    { name_matches ? <FormatedText keyword={props.name} term={props.term} /> : props.name }
-                    { !name_matches && !header_matches && props.keyword && <span className="hint">{_("Contains:")} <FormatedText keyword={props.keyword} term={props.term} /></span> }
+                    { props.header && <span className="hint">{header_matches ? <FormattedText keyword={props.header} term={props.term} /> : props.header}</span> }
+                    { name_matches ? <FormattedText keyword={props.name} term={props.term} /> : props.name }
+                    { !name_matches && !header_matches && props.keyword && <span className="hint">{_("Contains:")} <FormattedText keyword={props.keyword} term={props.term} /></span> }
                 </a>
                 {s && s.type && <PageStatus status={s} name={props.name} />}
                 { props.actions &&

pkg/storaged/client.js

@@ -436,7 +436,7 @@ function init_model(callback) {
     }
 
     function enable_nfs_features() {
-        // mount.nfs might be in */sbin but that ins't always in
+        // mount.nfs might be in */sbin but that isn't always in
         // $PATH, such as when connecting from CentOS to another
         // machine via SSH as non-root.
         const std_path = "PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin";