amenzhinsky
Repos
35
Followers
34

Azure IoT Hub SDK for Golang

46
43

D-Bus code-generation for golang

14
11

Run code from memory

240
30

Command line D-Bus inspector

4
0

PostgreSQL Partitioning Scripts

4
3

Events

closed issue
Is there a way to start the process with root privileges?
Created at 4 days ago
issue comment
Is there a way to start the process with root privileges?

To start a process using this package with root privileges you'd need to start your program as root.

Created at 4 days ago
issue comment
Add go 1.19 support

@guelfey done

Created at 1 month ago

freebsd: Remove the cgo dependency from transport_unixcred_freebsd.go

This also stops the code from wrongly using sizeof(struct ucred) which is unrelated to SCM_CREDS. The size of the correct cmsgcreds structure is hard-coded here - this ABI has not changed for ~25 years.

The added dependency on golang.org/x/sys/unix could be avoided if necessary, e.g. by using unsafe.Sizeof(uintptr).

Fixes #331

fix(ci): apt-get update before installing dbus

fix(ci): skip .git in codespell

Merge pull request #342 from godbus/fix/ci_update

fix(ci): apt-get update before installing dbus

Merge pull request #332 from dfr/freebsd-unixcred

freebsd: Remove the cgo dependency from transport_unixcred_freebsd.go

Add go 1.19 support

Created at 1 month ago

Clean up comments.

Merge pull request #71 from dangerousHobo/master

Clean up comments.

Created at 1 month ago
pull request closed
Clean up comments.

Minor fixes to comments.

Created at 1 month ago

Bugfix/add len check to signal lookup (#1)

  • Add len check to signal lookup

Co-authored-by: Oscar Postlethwaite oscar.a.postlethwaite@dyson.com

Merge pull request #13 from scriptinglife1/master

Add len check to signal lookup

Created at 1 month ago
pull request closed
Add len check to signal lookup

I've had issues with our code crashing when trying to lookup signals which don't match the expected format. I've added a len check to the code which types the signal to verify the list of args is long enough before trying to type them.

Created at 1 month ago
pull request closed
Handle semicolons in MQTT topic property.

Addresses issue #68. Allows semicolons to continue to exist in the properties and not trip up url.ParseQuery.

Created at 1 month ago

Handle semicolons in MQTT topic property.

Addresses issue #68.

Merge pull request #69 from dangerousHobo/handle-semicolon-in-mqtt-topic-property

Handle semicolons in MQTT topic property.

Created at 1 month ago
closed issue
%1 is not a valid Win32 application.

here is my code

package main

import (
	_ "embed"
	"fmt"
	"log"

	"github.com/amenzhinsky/go-memexec"
)

func main() {

	// go:embed ./DSGrab.exe
	var mybinary []byte

	exe, err := memexec.New(mybinary)
	if err != nil {
		log.Fatalln(err)
	}
	defer exe.Close()

	cmd := exe.Command()
	fmt.Println(cmd.CombinedOutput()) // cmd is a `*exec.Cmd` from the standard library
}

DELETED

The error i get is:

[] fork/exec C:\Users\ghost\AppData\Local\Temp\go-memexec-3129038014.exe: %1 is not a valid Win32 application.
Created at 1 month ago
issue comment
%1 is not a valid Win32 application.

I suppose this is a scam issue looking at the author's github activity.

Created at 1 month ago
pull request opened
Add go 1.19 support

Added go1.19 to the testing matrix and gofmt'ed whole project, plus updated the setup go step to v3

Created at 1 month ago
create branch
amenzhinsky create branch add-go1.19-support
Created at 1 month ago

Add go 1.19 support

Created at 1 month ago

tests: add a test for closing a signal channel after calling RemoveSignal

This should be safe, but currently fails if any goroutines have been spun up due to the channel buffer filling.

conn: ensure that no signals will be sent on a channel after RemoveSignal

Ran gofmt/goimports

Fixed formatting and ordered imports to prevent future merge conflicts.

Fix DBUS_SYSTEM_BUS_ADDRESS handling

The dbus specification says that the environment variable must already contain the transport prefix: https://dbus.freedesktop.org/doc/dbus-specification.html#addresses

The old code would prevent the following use cases:

  • set DBUS_SYSTEM_BUS_ADDRESS to "unix:path=/var/run/dbus/special_system_bus_socket" which is how it has to be set for e.g. glib dbus to be able to connect. This would lead to getSystemBusPlatformAddress returning a broken "unix:path=unix:path=/var/run/dbus/special_system_bus_socket".
  • set DBUS_SYSTEM_BUS_ADDRESS to "unixexec:path=…", "tcp:host=…" or any other transport.

With this change we fix the behavior by using the environment variable as is instead of always prepending "unix:path=".

Fix: remove import of unused fmt package.

Merge branch 'master' into format-fixes

Simplify code

A bunch of simple code cleanups that should have no side-effects. Just a bit nicer to read/parse.

Add missing error checks

Seems like they have just been forgotten.

Avoid unnecessary conversions

Those vars are already of the right type.

Merge pull request #165 from muesli/format-fixes

Ran gofmt/goimports

tests: explicitly disable AppArmor support in nonce-tcp test

travis.yml: switch to the xenial VM image

Use Ubuntu Bionic and latest Go versions.

Update go.mod to v6

Update go.mod to v5

Fix v5 subpackages

Disable implicit module updates

Support last three go versions

Fix link to gnome developer documentation.

Fixed typos in godoc comments

Just a bunch of nitpicky typo fixes.

Created at 1 month ago
issue comment
Doesn't accept connection string without shared access key?

Hi, for now connection string parsing applies only to SAS authentication, if you need x509 try iotdevice.NewFromX509Cert.

Probably we can add x509 support when x509=true, but SDK needs to know where from it gets key/cert pair.

Created at 1 month ago
Add grafana_organization_user module

@rrey-aviatrix sorry for the delay, all done

Created at 2 months ago

Add grafana_organization_user module

Resolve review issues

Add changelog fragment

Remove unused code

Created at 2 months ago

Remove unused code and invenrory

Created at 2 months ago

url encode user email on get_user_id_from_mail

url encode login on get_user_from_login

add changelog fragment

test(team-and-user): add tests with encoded char in login/name

Ref: #264

build(1.5.2): release 1.5.2

ci(python): skip tests with Python 3.8 on devel

fix(pep8): fix sanity error git st

test(datasource): Add unit tests for grafana_datasource module

Adding these tests aims at being capable to detect drifts in the sent requests when one is refactoring the module.

Update tested Grafana versions

doc(readme): Update readme with relevant info about release

ci(actions): use ansible-test github action for sanity and units

Update tested Grafana versions

Merge branch 'main' into organization-user-module

Created at 2 months ago
pull request closed
Rewrite context watcher to use single background goroutine

This removes the need to start a new gouroutine each query supplied with a non-context.Background context, instead it maintains a background routine that handles all Watch requests.

This can save some cpu cycles for applications that pass down net/http.(*Request).Context or grpc-generated Context to every single sql query.

Here is benchcmp for ctxwatch package:

benchmark                                  old ns/op     new ns/op     delta
BenchmarkContextWatcherUncancellable-8     27.3          13.6          -50.18%
BenchmarkContextWatcherCancelled-8         599           454           -24.20%
BenchmarkContextWatcherCancellable-8       420           319           -24.04%

benchmark                                  old allocs     new allocs     delta
BenchmarkContextWatcherUncancellable-8     0              0              +0.00%
BenchmarkContextWatcherCancelled-8         4              3              -25.00%
BenchmarkContextWatcherCancellable-8       1              0              -100.00%

benchmark                                  old bytes     new bytes     delta
BenchmarkContextWatcherUncancellable-8     0             0             +0.00%
BenchmarkContextWatcherCancelled-8         208           176           -15.38%
BenchmarkContextWatcherCancellable-8       32            0             -100.00%

Unfortunately go benchmarks don't include info about goroutines used since they're handled by the runtime.

Created at 2 months ago

Fix data race

Created at 2 months ago
pull request opened
Rewrite context watcher to use single background goroutine

This removes the need to start a new gouroutine each query supplied with a non-context.Background context, instead it maintains a background routine that handles all Watch requests.

Applications that pass down net/http.(*Request).Context or grpc-generated Context to every query this can save some cpu cycles.

Here is benchcmp for ctxwatch package:

benchmark                                  old ns/op     new ns/op     delta
BenchmarkContextWatcherUncancellable-8     27.3          13.6          -50.18%
BenchmarkContextWatcherCancelled-8         599           454           -24.20%
BenchmarkContextWatcherCancellable-8       420           319           -24.04%

benchmark                                  old allocs     new allocs     delta
BenchmarkContextWatcherUncancellable-8     0              0              +0.00%
BenchmarkContextWatcherCancelled-8         4              3              -25.00%
BenchmarkContextWatcherCancellable-8       1              0              -100.00%

benchmark                                  old bytes     new bytes     delta
BenchmarkContextWatcherUncancellable-8     0             0             +0.00%
BenchmarkContextWatcherCancelled-8         208           176           -15.38%
BenchmarkContextWatcherCancellable-8       32            0             -100.00%

Unfortunately go benchmarks don't include info about goroutines used since they're handled by the runtime.

Created at 2 months ago
create branch
amenzhinsky create branch goroutineless-context-watcher
Created at 2 months ago

Rewrite context watcher to use single goroutine instead of one per each watch

Created at 2 months ago
amenzhinsky create tag v0.4.2
Created at 2 months ago

Add client scrape benchmarks

Created at 2 months ago