47 Commits (e72ff5dd43db1e58ec52d7ac2f7ab5b20f4054ba)

Author	SHA1	Message	Date
Frank Schröder	12216583a1	New config parser, HCL support, multiple bind addrs (#3480) ... * new config parser for agent This patch implements a new config parser for the consul agent which makes the following changes to the previous implementation: * add HCL support * all configuration fragments in tests and for default config are expressed as HCL fragments * HCL fragments can be provided on the command line so that they can eventually replace the command line flags. * HCL/JSON fragments are parsed into a temporary Config structure which can be merged using reflection (all values are pointers). The existing merge logic of overwrite for values and append for slices has been preserved. * A single builder process generates a typed runtime configuration for the agent. The new implementation is more strict and fails in the builder process if no valid runtime configuration can be generated. Therefore, additional validations in other parts of the code should be removed. The builder also pre-computes all required network addresses so that no address/port magic should be required where the configuration is used and should therefore be removed. * Upgrade github.com/hashicorp/hcl to support int64 * improve error messages * fix directory permission test * Fix rtt test * Fix ForceLeave test * Skip performance test for now until we know what to do * Update github.com/hashicorp/memberlist to update log prefix * Make memberlist use the default logger * improve config error handling * do not fail on non-existing data-dir * experiment with non-uniform timeouts to get a handle on stalled leader elections * Run tests for packages separately to eliminate the spurious port conflicts * refactor private address detection and unify approach for ipv4 and ipv6. Fixes #2825 * do not allow unix sockets for DNS * improve bind and advertise addr error handling * go through builder using test coverage * minimal update to the docs * more coverage tests fixed * more tests * fix makefile * cleanup * fix port conflicts with external port server 'porter' * stop test server on error * do not run api test that change global ENV concurrently with the other tests * Run remaining api tests concurrently * no need for retry with the port number service * monkey patch race condition in go-sockaddr until we understand why that fails * monkey patch hcl decoder race condidtion until we understand why that fails * monkey patch spurious errors in strings.EqualFold from here * add test for hcl decoder race condition. Run with go test -parallel 128 * Increase timeout again * cleanup * don't log port allocations by default * use base command arg parsing to format help output properly * handle -dc deprecation case in Build * switch autopilot.max_trailing_logs to int * remove duplicate test case * remove unused methods * remove comments about flag/config value inconsistencies * switch got and want around since the error message was misleading. * Removes a stray debug log. * Removes a stray newline in imports. * Fixes TestACL_Version8. * Runs go fmt. * Adds a default case for unknown address types. * Reoders and reformats some imports. * Adds some comments and fixes typos. * Reorders imports. * add unix socket support for dns later * drop all deprecated flags and arguments * fix wrong field name * remove stray node-id file * drop unnecessary patch section in test * drop duplicate test * add test for LeaveOnTerm and SkipLeaveOnInt in client mode * drop "bla" and add clarifying comment for the test * split up tests to support enterprise/non-enterprise tests * drop raft multiplier and derive values during build phase * sanitize runtime config reflectively and add test * detect invalid config fields * fix tests with invalid config fields * use different values for wan sanitiziation test * drop recursor in favor of recursors * allow dns_config.udp_answer_limit to be zero * make sure tests run on machines with multiple ips * Fix failing tests in a few more places by providing a bind address in the test * Gets rid of skipped TestAgent_CheckPerformanceSettings and adds case for builder. * Add porter to server_test.go to make tests there less flaky * go fmt	7 years ago
Kyle Havlovitz	2ada0439d4	Add rpc_listener option to segment config	7 years ago
Illirgway	e0cc1ce679	Fix bug with unused (replaced with "") CONSUL_HTTP_AUTH in some places ... example: https://github.com/hashicorp/consul/blob/master/watch/plan.go#L26 conf := consulapi.DefaultConfig() conf.Address = address conf.Datacenter = p.Datacenter conf.Token = p.Token # <-- replace Token from DefaultConfig/CONSUL_HTTP_AUTH with "" client, err := consulapi.NewClient(conf) how to reproduce bug: 0. consul -> localhost:8500 with more than 0 service checks 1. deny all for anonymous token 2. create appropriate acl <token> for watch checks (agent:read + node:read,service:read) 3. bash: CONSUL_HTTP_AUTH=<token> consul watch -http-addr=localhost:8500 -type=checks # --> return [] consul watch -http-addr=localhost:8500 -type=checks -token=<token> # -> return { .... right json result .... }	7 years ago
James Phillips	8d0a048408	Expands and rework context support in the API client. (#3273)	7 years ago
Preetha Appan	9eaf56bfe3	Fix socket file handle leaks from old blocking queries upon consul reload. This fixes issue #3018	8 years ago
James Phillips	3baa703d02	Merge pull request #3077 from jwasinger/issue-2850 ... Prevent Session.Destroy from leaving open TCP connections	8 years ago
Jared Wasinger	d2ef61d079	Prevent Session.Destroy from leaving open TCP connections	8 years ago
Kyle Havlovitz	4e44db87e3	Don't overwrite Transport's TLS config if it's been set	8 years ago
Kyle Havlovitz	04f1ff78f8	Remove superfluous default HttpClient check in NewClient	8 years ago
Frank Schroeder	3e00e36f41	golint: Untangle if blocks with return in else	8 years ago
Kyle Havlovitz	c6d0b4cf90	Make the API client's httpClient more pluggable (#2926)	8 years ago
Kyle Havlovitz	ae6bf56ee1	Add tls client options to api/cli	8 years ago
Jeff Mitchell	44733caf5c	Update unix dial functions to use DialContext with new go-cleanhttp	8 years ago
Kyle Havlovitz	c9abafab92	Update docs and give better error for unknown client scheme	8 years ago
Kyle Havlovitz	a7a6ee9f4c	Allow prefixing -http-addr with http/https schemes	8 years ago
Kyle Havlovitz	bad037fb58	Cleanup and formatting adjustments	8 years ago
Kyle Havlovitz	abdf1fbab3	Convert keyring command to use base.Command	8 years ago
Kyle Havlovitz	87c0283bb1	Update client api and docs for node metadata	8 years ago
James Phillips	c744792fc4	Merge pull request #2238 from hasyimibhar/master ... Remove duplicated environment variables	8 years ago
James Phillips	9f7a973ace	Adds an `X-Consul-Translate-Addresses` to signal translation is enabled.	8 years ago
Hasyimi Bahrudin	105cea9bae	Add api environment variables as constants for consistency	8 years ago
Hasyimi Bahrudin	bb6bb19aad	Remove redundant hardcoded environment variables ... The following hardcoded environment variables are removed: * CONSUL_RPC_ADDR * CONSUL_HTTP_ADDR	8 years ago
Jeff Mitchell	f1a67836d8	Use header to send Consul token rather than query param.	8 years ago
James Phillips	0eb6279c5e	Improves the comment for the Address field.	9 years ago
James Phillips	1cfed981b3	Adds TLS config helper to API client.	9 years ago
James Phillips	abacccd6e5	Switches default for API client to pooled connections.	9 years ago
James Phillips	58fb27e70f	Factors server error checking into a new function.	9 years ago
James Phillips	c048c5aca6	Makes the API behave better with small wait values.	9 years ago
James Phillips	ebff3fe0d0	Builds the insecure SSL transport off the clean default transport.	9 years ago
Jeff Mitchell	4604af6aa5	Revert "Adds client and transport pooling in the API so we don't leak connections."	9 years ago
James Phillips	f6546a0e46	Makes the insecure transport work like the default one.	9 years ago
James Phillips	d57e60a123	Adds client and transport pooling in the API so we don't leak connections.	9 years ago
James Phillips	787f946f8d	Adds support for coordinates to client API.	9 years ago
Jeff Mitchell	1e3840b044	Update cleanhttp repo location	9 years ago
Jeff Mitchell	9a5fd5424a	Use cleanhttp to get rid of DefaultTransport	9 years ago
Jeff Mitchell	f49fc095ef	Don't use http.DefaultClient ... Two of the changes are in tests; the one of consequence is in the API. As explained in #1308 this can cause conflicts with downstream programs. Fixes #1308.	9 years ago
Dale Wijnand	5a28ebcaa3	Fix a bunch of typos.	9 years ago
Seth Vargo	b4f8c98fbb	Read select environment variables when generating the default configuration	10 years ago
Armon Dadgar	5f627fca1b	api: Refactoring into shared write logic	10 years ago
Armon Dadgar	c0c9fb58ce	api: Refactoring into shared query logic	10 years ago
Ryan Uber	b30af9536d	Merge pull request #608 from imkira/fixBodyLeak ... fixed: body not closed for non HTTP 200 responses	10 years ago
Mário Freitas	f45105383c	fixed: requireOK should close Body on error	10 years ago
Ryan Uber	4675cdf01c	agent: beginning refactor	10 years ago
Jeff Mitchell	9fcea08dbb	Ensure a socket is created for permissions adjustment tests and fix some ... items pointed out in the code review This code is copyright 2014 Akamai Technologies, Inc. <opensource@akamai.com>	10 years ago
Jeff Mitchell	11a3ce0bdd	RPC and HTTP interfaces fully generically-sockified so Unix is supported. ... Client works for RPC; will honor CONSUL_RPC_ADDR. HTTP works via consul/api; honors CONSUL_HTTP_ADDR. The format of a Unix socket in configuration data is: "unix://[/path/to/socket];[username or uid];[gid];[mode]" Obviously, the user must have appropriate permissions to create the socket file in the given path and assign the requested uid/gid. Also note that Go does not support gid lookups from group name, so gid must be numeric. See https://codereview.appspot.com/101310044 When connecting from the client, the format is just the first part of the above line: "unix://[/path/to/socket]" This code is copyright 2014 Akamai Technologies, Inc. <opensource@akamai.com>	10 years ago
Ryan Uber	60ebd63760	Import HTTP basic auth patch from armon/consul-api#16	10 years ago
Ryan Uber	5555e0eb9b	api: initial import from armon/consul-api	10 years ago